Skip to content

[fix] Export pkg-config flags for cuda and hwloc - [merged] #157

New issue

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

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

Already on GitHub? Sign in to your account

Closed
iskra-anl opened this issue Apr 20, 2021 · 10 comments
Closed

[fix] Export pkg-config flags for cuda and hwloc - [merged] #157

iskra-anl opened this issue Apr 20, 2021 · 10 comments

Comments

@iskra-anl
Copy link
Contributor

In GitLab by @NicolasDenoyelle on Oct 20, 2020, 12:02

Merges pkg_config_flags -> staging

pkg-config --libs aml and pkg-config --cflags aml now gives the correct set of flags.

Fix #74

@iskra-anl
Copy link
Contributor Author

In GitLab by @NicolasDenoyelle on Oct 20, 2020, 12:05

@perarnau please check this MR.

I tested with options --with-cuda and --with-cuda=no. It is working and flags look ok to me.

Note that it would be nice to setup a CI stage where we try to build a dummy app linked with aml.

@iskra-anl
Copy link
Contributor Author

In GitLab by @perarnau on Oct 20, 2020, 12:18

Commented on configure.ac line 169

Already done a few lines below ?

@iskra-anl
Copy link
Contributor Author

In GitLab by @perarnau on Oct 20, 2020, 12:39

Note that it would be nice to setup a CI stage where we try to build a dummy app linked with aml.

I agree, installcheck is very tricky to figure out in general, but maybe we can find a way to leverage the nix environment for that.

@iskra-anl
Copy link
Contributor Author

In GitLab by @perarnau on Oct 20, 2020, 12:39

changed the description

@iskra-anl
Copy link
Contributor Author

In GitLab by @NicolasDenoyelle on Oct 20, 2020, 14:47

Commented on configure.ac line 169

changed this line in version 2 of the diff

@iskra-anl
Copy link
Contributor Author

In GitLab by @NicolasDenoyelle on Oct 20, 2020, 14:48

added 1 commit

  • 500dcec - [fix] Export pkg-config flags for cuda and hwloc

Compare with previous version

@iskra-anl
Copy link
Contributor Author

In GitLab by @NicolasDenoyelle on Oct 20, 2020, 14:48

@perarnau ready when the pipeline goes through.

@iskra-anl
Copy link
Contributor Author

In GitLab by @perarnau on Oct 20, 2020, 14:55

resolved all threads

@iskra-anl
Copy link
Contributor Author

In GitLab by @perarnau on Oct 20, 2020, 14:55

merged

@iskra-anl
Copy link
Contributor Author

In GitLab by @perarnau on Oct 20, 2020, 14:55

mentioned in commit 2910ba2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants