[Bug 1324590] Review Request: hfi1-psm - Intel PSM Libraries

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1324590



--- Comment #34 from Michal Schmidt <mschmidt@xxxxxxxxxx> ---
(In reply to paul.j.reger from comment #31)
> As such, I have changed that version to 10.1.0. But, that version is subject
> to change 

I'm confused. Do you mean that this is just a pre-release of 10.1.0 and a final
10.1.0 is yet to be released?
Or you're not yet sure what version number the real release will have?

> as the code is not quite ready to be released.

I think we have different ideas what it means to release something.
It is tagged in the public git repo, a tarball was made from it and published
for anyone to download => it is released.
Do you mean this release does not have production-level quality? Consider using
common terms like "alpha", "beta", "rc" in the version string of such releases,
or have a well-defined numeric versioning scheme (for example: X.Y.Z where Y is
an odd number are pre-releases).
Consider writing release announcements to a mailing list (I see
intel-opa@xxxxxxxxxxxx is mentioned on https://github.com/01org/opa-psm2/wiki,
but so far there were only test messages) and/or post them on a webpage.
Whatever versioning scheme you adopt, make it understandable for your
downstream consumers and distributors.

> I will push the new version of the spec file for your review.
> 
> Can you please re-review my spec file, and make further comments?

I will.

> Or, if you want, we can abandon this review, as Doug suggests,

I did not read that as a suggestion to abandon this review.
We all still would like to get hfi1-psm accepted into Fedora, so the package
needs to pass through the review.

> and just review a tar ball.

Did you create this tarball specially for the purpose of this review? That's
not what Doug was asking for. Neither was to upload tarballs to Bugzilla.
Just make sure the tarballs you publish on github use a sane naming and
versioning scheme.

This tarball can be acceptable as a base for a Fedora package:
https://github.com/01org/opa-psm2/releases/download/10_1/hfi1-psm-10.1-0.tar.gz
If I were the packager, I'd infer from the file name the version of the
software is "10.1" and "-0" is some superfluous string to ignore.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
http://lists.fedoraproject.org/admin/lists/package-review@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]