Skip to content
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

pacman: export CC and CXX variables explicitly #2501

Merged
merged 1 commit into from
May 19, 2021

Conversation

MehdiChinoune
Copy link
Collaborator

No description provided.

@jeremyd2019
Copy link
Member

I thought of this, but didn't know what if anything it might break. I do know that python appreciates knowing that the compiler is named clang, and turns on proper support for clang pgo/lto, so I set those vars explicitly in python's PKGBUILD

@Biswa96
Copy link
Member

Biswa96 commented May 14, 2021

This seems to be same change as the other PR in mingw-packages. What happen if some projects use clang even in mingw64 or mingw32? e.g. zig, ghld etc. Why do we need this change?

@Alexpux
Copy link
Member

Alexpux commented May 15, 2021

Setting this variables in conf files will not export them to build. You must also export this variables in /usr/share/makepkg/buildenv/buildflags.sh

@MehdiChinoune MehdiChinoune force-pushed the default-compilers branch 6 times, most recently from bebf9d2 to deb0613 Compare May 15, 2021 07:33
@jeremyd2019

This comment has been minimized.

@lazka lazka merged commit 53ec56b into msys2:master May 19, 2021
@MehdiChinoune MehdiChinoune deleted the default-compilers branch May 19, 2021 06:09
lazka added a commit to msys2/msys2-pacman that referenced this pull request May 23, 2021
lazka added a commit to msys2/msys2-pacman that referenced this pull request Jun 3, 2021
lazka added a commit to msys2/msys2-pacman that referenced this pull request Sep 4, 2021
ghost pushed a commit to ookiineko-cygport/cygwin-pacman that referenced this pull request Dec 14, 2022
ghost pushed a commit to ookiineko-cygport/cygwin-pacman that referenced this pull request Dec 15, 2022
ghost pushed a commit to ookiineko-cygport/cygwin-pacman that referenced this pull request Dec 16, 2022
ghost pushed a commit to ookiineko-cygport/cygwin-pacman that referenced this pull request Dec 16, 2022
lazka added a commit to msys2/msys2-pacman that referenced this pull request Feb 27, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request Feb 28, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request Feb 28, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request Feb 28, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request Feb 28, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request Mar 1, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request Mar 1, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request Mar 2, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request Mar 2, 2023
lazka added a commit to msys2/msys2-pacman that referenced this pull request May 5, 2024
lazka added a commit to lazka/msys2-pacman that referenced this pull request May 5, 2024
lazka added a commit to lazka/msys2-pacman that referenced this pull request May 5, 2024
lazka added a commit to lazka/msys2-pacman that referenced this pull request May 5, 2024
lazka added a commit to msys2/msys2-pacman that referenced this pull request May 5, 2024
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.

5 participants