PR#376: Mock: fix dist tag detection for el7/8 - FedoraReview

FedoraReview

#376 Mock: fix dist tag detection for el7/8

Merged 4 years ago by ngompa. Opened 4 years ago by ret2libc.

ret2libc/FedoraReview fix-match-tag into master

Mock: fix dist tag detection for el7/8

Riccardo Schirone • 4 years ago

bc89706

src/FedoraReview/mock.py

file modified

+1 -1

		`@@ -68,7 +68,7 @@`
		`releases = [r.rsplit(".", 2)[0] for r in releases]`
		`if not len(set(releases)) == 1:`
		`return ""`
		`- match = re.search("(fc\|el)[0-9][0-9]", releases[0])`
		`+ match = re.search("(fc\|el)[0-9][0-9]?", releases[0])`
		`return match.group() if match else ""`

ret2libc commented 4 years ago

When using prebuilt packages with "el8." disttag, they are not correctly identified because the regexp looks for two digits. This should make it work for both fcXX and elY tags.

Metadata

Assignee

None

Tags

No Tags

Changes Summary 1

+1 -1

file changed

src/FedoraReview/mock.py

FedoraReview

Source Code

#376 Mock: fix dist tag detection for el7/8 Merged 4 years ago by ngompa. Opened 4 years ago by ret2libc. ret2libc/FedoraReview fix-match-tag into master

Metadata

Changes Summary 1

#376 Mock: fix dist tag detection for el7/8

Merged 4 years ago by ngompa. Opened 4 years ago by ret2libc.

ret2libc/FedoraReview fix-match-tag into master