Skip to content

Fix use of DESTDIR with PREFIX in meson#2726

Open
ThmsP wants to merge 1 commit intosu2code:developfrom
ThmsP:fix_destdir_prefix
Open

Fix use of DESTDIR with PREFIX in meson#2726
ThmsP wants to merge 1 commit intosu2code:developfrom
ThmsP:fix_destdir_prefix

Conversation

@ThmsP
Copy link

@ThmsP ThmsP commented Feb 6, 2026

Proposed Changes

I had to package SU2 by rpm. Doing so I used DESTDIR and PREFIX :

./meson.py build --prefix=%{_prefix}
DESTDIR=%{buildroot} ./ninja -C build install

Everything works fine except I had to do a small fix in this file :SU2_PY/pySU2/install.sh

PR Checklist

Put an X by all that apply. You can fill this out after submitting the PR. If you have any questions, don't hesitate to ask! We want to help. These are a guide for you to know what the reviewers will be looking for in your contribution.

  • I am submitting my contribution to the develop branch.
  • My contribution generates no new compiler warnings (try with --warnlevel=3 when using meson).
  • My contribution is commented and consistent with SU2 style (https://su2code.github.io/docs_v7/Style-Guide/).
  • I used the pre-commit hook to prevent dirty commits and used pre-commit run --all to format old commits.
  • I have added a test case that demonstrates my contribution, if necessary.
  • I have updated appropriate documentation (Tutorials, Docs Page, config_template.cpp), if necessary.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant