koji

#4265 Drop explicit cvs requires?
Closed: Fixed by tkopecek. Opened by mikem.

Closed: Fixed
Reopen Issue

The builder requires cvs in case anyone is building using cvs. At this point it seems odd to make this a fixed requires. I don't think we need to drop the cvs support necessarily, but if an instance needs to use cvs, they can just make sure it's installed.

See also: https://bugzilla.redhat.com/show_bug.cgi?id=2319897#c10

+1

Metadata Update from @tkopecek:
- Custom field Size adjusted to None
romaingeissler1a commented

Hi,

So, has a consensus been reached here ? Or we need to let a bit more time for other maintainers/contributors to voice their opinion ?

Metadata Update from @tkopecek:
- Issue set to the milestone: 1.36

I think it is a safe change.
romaingeissler1a commented

I have submitted https://pagure.io/koji/pull-request/4271 to implement this proposal.

Metadata Update from @mikem:
- Issue tagged with: no_qe

This is marked for 1.36, but I think it should be fine to move to 1.35.2 since the change is ready. Any objections @tkopecek ?

Metadata Update from @tkopecek:
- Issue set to the milestone: 1.35.2 (was: 1.36)

Commit cd25b8dd fixes this issue

Commit fd026e73 relates to this ticket

Commit af275200 fixes this issue

This issue has been migrated to Fedora Forge:
https://forge.fedoraproject.org/koji/koji/issues/4265

Please continue any further discussion there.
Metadata
None

no_qe

None
None
Low
None
Related Pull Requests
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.