koji

#2784 Tolerate floats in metadata timestamps
Merged by tkopecek. Opened by alexi.
alexi/koji issue_2783  into  master

Download 2784.patch

Fixes #2783.

:thumbsup:

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

#2785 has already been merged, so I guess this can be closed?
Edited by alexi

No, I've just closed mine without merging as I've noticed yours.

Anyway, I was thinking about just float conversion. We can miss some updates (very hypothetically). If I've 123 timestamp for koji's repo and and external repo is 123.45 it will be ignored.

I added the round in case anything else depended on the timestamp being an int, although it didn't look like it would be an issue. Anyway, float(child.text) should be fine.

Metadata Update from @mfilip:
- Pull-request tagged with: testing-done

Commit d8c46e85 fixes this pull-request

Pull-Request has been merged by tkopecek
Metadata
None

testing-done testing-ready

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.