Skip to content

Fix travis#88

Merged
pablogsal merged 1 commit intopegenfrom
travis
Apr 17, 2020
Merged

Fix travis#88
pablogsal merged 1 commit intopegenfrom
travis

Conversation

@pablogsal
Copy link
Copy Markdown

No description provided.

@pablogsal pablogsal force-pushed the travis branch 12 times, most recently from f12a027 to 2fd3395 Compare April 17, 2020 20:32
BUILDPYTHON= python$(BUILDEXE)

PYTHON_FOR_REGEN=@PYTHON_FOR_REGEN@
PYTHON_FOR_REGEN?=@PYTHON_FOR_REGEN@
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just out of curiosity, what does the @...@ notation do?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It substitutes the values that autotools calculate in the configure step. Basically this file is a template that will be used to generate the real makefile after configure analyses the system.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation!

@pablogsal pablogsal changed the title WIP: Fix travis Fix travis Apr 17, 2020
@pablogsal pablogsal merged commit b625797 into pegen Apr 17, 2020
@pablogsal pablogsal deleted the travis branch April 17, 2020 21:32
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.

2 participants