|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago
Those examples are now outdated. My suggestion would be to import this page into the new docs repo too, and update it at the same time. | ||
tibbs commented 4 years ago That is unrelated to this change. I mentioned the need to work on these examples in the open ticket for this issue. | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago
Why describe Release tag before Version tag? I think the order should be switched. | ||
tibbs commented 4 years ago Because that puts the two shortest sections at the front, so users who want to know what goes in the release field don't have to skip the description of the Version: field. | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago
That's probably not related directly to this change, but: maybe this should be relaxed to say "MUST be incremented whenever the package is rebuilt in koji" (or something to that effect). People generally don't follow this guideline as written. | ||
tibbs commented 4 years ago Unrelated to this change. Neverless, the guideline is correct. Change version, reset release. It's pretty simple, and I do think most people follow it. You wouldn't change Version and leave Release: at 27, only setting it back to one the moment you want to build. | ||
|
||
|
||
|
||
zbyszek commented 4 years ago typo: pacakge | ||
|
||
zbyszek commented 4 years ago
... rebuilding the package | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago typo: must → most | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago typo: prerelasee ... or a snapshot | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago
Is this really necessary? Including the date is additional work that doesn't really bring that much benefit for many cases (*). (*) For example, systemd "upstream" is a snapshot of a branch with backported patches (as in https://github.com/systemd/systemd-stable/commits/v242-stable), there is no meaningful "date", because the time when the patch was written is non-monotonous, so cannot be used here, and the time when the patch was backported is not particularly interesting, and the time when the snapshot was packaged doesn't say anything about the contents. | ||
tibbs commented 4 years ago This is unrelated to the change in question. Please bring it up separately. | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago typo: previse | ||
|
||
|
||
|
||
|
||
|
||
|
||
zbyszek commented 4 years ago Maybe drop the part about CVS? It'll only confuse anyone below 30. | ||
tibbs commented 4 years ago Unrelated to this change. Besides, it's 100% valid, isn't it? Feel free to open a separate ticket about it if you like. | ||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
|
||
This is an initial pass at converting the last draft of the tilde+caret versioning scheme I had in the wiki to an actual guideline page.
Unfortunately because the RPM support for the new scheme is rawhide-only, we will have to keep both pages around for potentially quite some time. Also note that the examples page in the wiki will need even more work (as it was never finished and then never updated after we started allowing tildes). No idea when I might be able to get to that.
Finally, I haven't put in sufficient thought into converting from the old guidelines to the new. There is a section on it but I'm not yet completely sure that it's accurate.
typo