Skip to content

Change CPPEXTPATH to CPPPATH for mesa_private_inc_paths #106416

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TCROC
Copy link
Contributor

@TCROC TCROC commented May 14, 2025

Just another PR to consider alongside #106413.

Fixes: #106376

Hopefully this doesn't bring back the warnings that the other PR does and we get to have our cake and eat it to! :)

@TCROC
Copy link
Contributor Author

TCROC commented May 14, 2025

@akien-mga CC

🤞 this one doesn't bring it all those warnings. And pray the cosmic rays don't ruin our day.

@TCROC
Copy link
Contributor Author

TCROC commented May 14, 2025

That's interesting... why don't I encounter these errors when building on my local?

@akien-mga
Copy link
Member

That's the same thing I did in #106413, minus some cosmetic cleanup. It does bring back the warnings sadly.

@akien-mga
Copy link
Member

That's interesting... why don't I encounter these errors when building on my local?

You probably don't build with warnings=extra werror=yes like the CI does.

@TCROC
Copy link
Contributor Author

TCROC commented May 14, 2025

Ah I see. I'll still test your PR tho to confirm that it works (or doesn't) :)

@akien-mga
Copy link
Member

To clarify, my PR has two commits:

  • The first is the same as yours, and introduced warnings
  • The second tries another approach by keeping CPPEXTPATH but changing the order

@TCROC
Copy link
Contributor Author

TCROC commented May 14, 2025

Ah I see... I'm testing the 2nd commit now then :). Was confused at first but that makes much more sense :)

@Calinou Calinou added this to the 4.5 milestone May 15, 2025
@TCROC
Copy link
Contributor Author

TCROC commented May 20, 2025

Hey @akien-mga. Which PR between ours are you wanting to go with? I don't mind which. I'm just not sure what to do to make the pipeline happy.

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

Successfully merging this pull request may close these issues.

3 participants