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

Fix parsing of %prep macros #356

Merged
merged 1 commit into from
Mar 19, 2024
Merged

Fix parsing of %prep macros #356

merged 1 commit into from
Mar 19, 2024

Conversation

nforro
Copy link
Member

@nforro nforro commented Mar 19, 2024

Parsing of %prep macros was incorrect. For instance %patches macro when used in %prep section was wrongly parsed as %patch macro. This commit fixes that.

RELEASE NOTES BEGIN

We have fixed an issue in %prep section processing. For instance, if the %patches macro appeared there, it would have been converted to %patch es, causing failure when executing %prep later.

RELEASE NOTES END

Parsing of %prep macros was incorrect. For instance %patches macro
when used in %prep section was wrongly parsed as %patch macro.
This commit fixes that.

Signed-off-by: Nikola Forró <[email protected]>
Copy link
Contributor

Build succeeded.
https://softwarefactory-project.io/zuul/t/packit-service/buildset/301ef04130734daf9ee83973bd96eed4

✔️ pre-commit SUCCESS in 2m 29s
✔️ specfile-tests-rpm-deps SUCCESS in 1m 14s
✔️ specfile-tests-pip-deps SUCCESS in 1m 10s

Copy link
Member

@lachmanfrantisek lachmanfrantisek left a comment

Choose a reason for hiding this comment

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

Thanks!

@nforro nforro added the mergeit Zuul, merge it! label Mar 19, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://softwarefactory-project.io/zuul/t/packit-service/buildset/917749a5fc6d491f8852183fa2850430

✔️ pre-commit SUCCESS in 2m 32s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit e9a63b6 into main Mar 19, 2024
21 of 32 checks passed
@delete-merged-branch delete-merged-branch bot deleted the prep branch March 19, 2024 11:57
nforro added a commit to nforro/authselect that referenced this pull request Mar 20, 2024
It contains a specfile fix [1] that prevents breaking `%prep`
in authselect.spec and is not yet in production.

[1] packit/specfile#356

Signed-off-by: Nikola Forró <[email protected]>
nforro added a commit to nforro/authselect that referenced this pull request Mar 20, 2024
It contains a specfile fix [1] that prevents breaking `%prep`
in authselect.spec and is not yet in production.

[1] packit/specfile#356

Signed-off-by: Nikola Forró <[email protected]>
nforro added a commit to nforro/authselect that referenced this pull request Mar 20, 2024
It contains a specfile fix [1] that prevents breaking `%prep`
in authselect.spec and is not yet in production.

[1] packit/specfile#356

Signed-off-by: Nikola Forró <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Zuul, merge it!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants