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

copr: Workaround for CVE-2022-24765 fix #2

Merged
merged 1 commit into from Jun 13, 2022
Merged

copr: Workaround for CVE-2022-24765 fix #2

merged 1 commit into from Jun 13, 2022

Conversation

nirs
Copy link
Member

@nirs nirs commented Jun 13, 2022

Copr runs "make srpm" in a directory not owned by the current user. This
breaks git commands and we get the wrong version number (e.g. 2.4.4-1)
instead of (2.4.4-0.timestamp.githash). The wrong version number break
users and OST, never getting the latest version.

Change-Id: Ib1e0e1f9324541b44988e47b3409a871d0c572ee
Signed-off-by: Nir Soffer nsoffer@redhat.com

Copr runs "make srpm" in a directory not owned by the current user. This
breaks git commands and we get the wrong version number (e.g. 2.4.4-1)
instead of (2.4.4-0.timestamp.githash). The wrong version number break
users and OST, never getting the latest version.

Change-Id: Ib1e0e1f9324541b44988e47b3409a871d0c572ee
Signed-off-by: Nir Soffer <nsoffer@redhat.com>
Copy link
Member

@michalskrivanek michalskrivanek left a comment

Choose a reason for hiding this comment

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

in other projects we usually do it in a separate target but that's all right, this will do just fine

@nirs nirs merged commit e0a7a04 into oVirt:master Jun 13, 2022
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.

None yet

3 participants