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

rename user-rebindable {LINK,COMP}FLAGS into OCB_{LINK,COMP}FLAGS #303

Merged
merged 1 commit into from
Jun 15, 2024

Conversation

gasche
Copy link
Member

@gasche gasche commented Sep 5, 2019

This should help avoid build failures due to unintential
redefined-variable capture, see #301.

(failure report by Samik Shrotriya)

@dbuenzli
Copy link
Collaborator

dbuenzli commented Sep 5, 2019

Except for the lack of a changes entry this looks good (also I suspect you did a git grep of these variables and replaced all of them, I didn't check this)

@hhugo
Copy link
Collaborator

hhugo commented May 29, 2024

@gasche, I think we should merge this one

This should help avoid build failures due to unintential
redefined-variable capture, see ocaml#301.

(failure report by Samik Shrotriya)
@hhugo hhugo force-pushed the avoid-makefile-env-collision branch from 5324278 to 3130462 Compare June 15, 2024 06:01
@hhugo hhugo merged commit 8764927 into ocaml:master Jun 15, 2024
20 checks passed
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.

3 participants