Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Added installation instructions for ArchLinux and FreeBSD, organized installation page slightly #279

Merged
merged 2 commits into from
May 7, 2019

Conversation

acxz
Copy link
Contributor

@acxz acxz commented May 6, 2019

No description provided.

@bluescarni
Copy link
Member

Thanks for the PR! I have restarted some travis builds which failed due to transient network errors.

See the `AUR helpers <https://wiki.archlinux.org/index.php/AUR_helpers` page on
the Arch Linux wiki for more info.

Note: To install pagmo with optional dependency support like nlopt or ipopt,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a question about this. Wouldn't it be better to have pagmo depend on all optional packages instead and enable them unconditionally in the build script for Arch linux?

Copy link
Contributor Author

@acxz acxz May 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see the reasoning behind that, but I don't want users to install packages that they have no need for. Let's say I have a package which has 100 optional dependencies, but I only need to use 1 of them. I don't want the hassle of the rest of the 99 packages. Rather I should build only based on what the current system has. They are optional for a reason and I would like to keep it that way, unless if they are required by the CMakelists.txt from the source and the build fails without them.

Although I would like the optional dependency check to happen inside the CMakelists instead of the package management level. But I understand that is against this project's policy, based on #276 .

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, fair enough!

@acxz acxz changed the title Added installtion instructions for ArchLinux and FreeBSD, organized installation page slightly Added installation instructions for ArchLinux and FreeBSD, organized installation page slightly May 7, 2019
@bluescarni
Copy link
Member

@acxz looks good to me. Can I merge or would you like to add something else?

@acxz
Copy link
Contributor Author

acxz commented May 7, 2019

I think it's good to go.

@bluescarni bluescarni merged commit c65d504 into esa:master May 7, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants