Plume/documentation
6
2
Fork 11
Code Issues 5 Pull requests 3 Releases Wiki Activity

need libpq-dev lib for rpm-based distros, fix deprecated cargo command usage #90

Manually merged
igalic merged 3 commits from clasick/documentation:dev into dev 2020-09-14 21:01:55 +00:00
Conversation 2 Commits 3 Files changed 2 +2 -2
clasick commented 2020-08-27 21:09:41 +00:00
Contributor
Copy link

There is an error when installing on Fedora (32) where during compilation, we can't find the libraries for postgresql.

note: /usr/bin/ld: cannot find -lpq

Installing libpq-dev using the command:

sudo dnf install libpq-dev

Fixes the error (even without a terminal or OS restart).

There is an error when installing on Fedora (32) where during compilation, we can't find the libraries for postgresql. ``` note: /usr/bin/ld: cannot find -lpq ``` Installing libpq-dev using the command: ``` sudo dnf install libpq-dev ``` Fixes the error (even without a terminal or OS restart).
👍 1
kiwii reviewed 2020-08-29 15:33:08 +00:00
source/installation/deps/rpm.html.md Outdated
@ -5,3 +5,3 @@
```bash
# If you want PostgreSQL
dnf install postgresql-server postgresql-contrib libpqxx libpqxx-devel git curl gcc make openssl openssl-devel gettext clang-devel
dnf install postgresql-server postgresql-contrib libpqxx libpqxx-devel libpq-devel lib git curl gcc make openssl openssl-devel gettext clang-devel
kiwii commented 2020-08-29 15:33:08 +00:00
Owner
Copy link

Is lib a real package, or is it a typo?

Also, are libpqxx and libpqxx-devel still required (I guess these are the C++ libs, but Plume may only need the C versions)?

Is `lib` a real package, or is it a typo? Also, are `libpqxx` and `libpqxx-devel` still required (I guess these are the C++ libs, but Plume may only need the C versions)?
clasick commented 2020-08-29 15:52:29 +00:00
Author
Contributor
Copy link

Oops, thanks for point that out, didn't realize that typo got in.

As for libpqxx and libpqxx-devel, after removing those 2 packages, what can I build to confirm whether they are still needed or not? Should I do front/back end, plm, and cli builds to make sure of this?

Oops, thanks for point that out, didn't realize that typo got in. As for `libpqxx` and `libpqxx-devel`, after removing those 2 packages, what can I build to confirm whether they are still needed or not? Should I do front/back end, plm, and cli builds to make sure of this?
igalic commented 2020-08-29 15:54:30 +00:00
Owner
Copy link

xx are generally for C++ and should not be needed

xx are generally for C++ and should not be needed
clasick commented 2020-08-29 21:43:35 +00:00
Author
Contributor
Copy link

@kiwii @igalic Can confirm now that the xx packages are not actually needed. Uninstalled both and was able to successfully run the build commands without any errors.

@kiwii @igalic Can confirm now that the xx packages are not actually needed. Uninstalled both and was able to successfully run the build commands without any errors.
clasick commented 2020-08-29 21:45:47 +00:00
Author
Contributor
Copy link

Added another to fix deprecated usage of cargo install. If the existing command as specified in the documentation is used, it throws this error:

[vkk@vkk Plume]$ cargo install --no-default-features --features postgres -f
error: Using `cargo install` to install the binaries for the package in current working directory is no longer supported, use `cargo install --path .` instead. Use `cargo build` if you want to simply build the package.

Added --path . parameter to fix this.

Added another to fix deprecated usage of `cargo install`. If the existing command as specified in the documentation is used, it throws this error: ``` [vkk@vkk Plume]$ cargo install --no-default-features --features postgres -f error: Using `cargo install` to install the binaries for the package in current working directory is no longer supported, use `cargo install --path .` instead. Use `cargo build` if you want to simply build the package. ``` Added `--path .` parameter to fix this.
clasick changed title from need libpq-dev lib for Fedora to need libpq-dev lib for rpm-based distros, fix deprecated cargo command usage 2020-08-29 21:46:37 +00:00
clasick commented 2020-09-14 19:12:09 +00:00
Author
Contributor
Copy link

@kiwii can we finish this pull request?

@kiwii can we finish this pull request?
igalic manually merged commit 6d02554fc7 into dev 2020-09-14 21:01:55 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something isn't working

  
dependencies

Pull requests that update a dependency file

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
wontfix

This will not be worked on

No labels
bug
dependencies
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: plume/documentation#90
No description provided.
Delete branch "clasick/documentation:dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?