https://bugzilla.redhat.com/show_bug.cgi?id=1907014
Bug ID: 1907014 Summary: Review Request: rust-cty - bindgen type aliases for C Product: Fedora Version: rawhide Hardware: All OS: Linux Status: NEW Component: Package Review Severity: medium Priority: medium Assignee: nobody@fedoraproject.org Reporter: dank@qemfd.net QA Contact: extras-qa@fedoraproject.org CC: package-review@lists.fedoraproject.org Target Milestone: --- Classification: Fedora
Spec URL: https://nick-black.com/tabpower/fedora-rust/rust-cty.spec SRPM URL: https://nick-black.com/tabpower/fedora-rust/rust-cty-0.2.1-1.fc32.src.rpm Description: Type aliases to C types like c_int for use with bindgen Fedora Account System Username: nickblack
Pretty much entirely rust2rpm boilerplate.
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
Fabio Valentini decathorpe@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |decathorpe@gmail.com Summary|Review Request: rust-cty - |Review Request: rust-cty - |bindgen type aliases for C |Type aliases to C types | |like c_int for use with | |bindgen Doc Type|--- |If docs needed, set a value
--- Comment #1 from Fabio Valentini decathorpe@gmail.com --- Please use the actual package Summary for the bug title.
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
--- Comment #2 from Nick Black dank@qemfd.net --- (In reply to Fabio Valentini from comment #1)
Please use the actual package Summary for the bug title.
Updated, thanks.
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
Igor Raits igor.raits@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |igor.raits@gmail.com
--- Comment #3 from Igor Raits igor.raits@gmail.com --- The %license is missing, otherwise looks good.
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
--- Comment #4 from Nick Black dank@qemfd.net --- Thanks Igor, I've updated the spec and SRPM. I've also made the SRPM against f34 this time.
https://nick-black.com/tabpower/fedora-rust/rust-cty.spec https://nick-black.com/tabpower/fedora-rust/rust-cty-0.2.1-1.fc34.src.rpm
PTAL
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
--- Comment #5 from Igor Raits igor.raits@gmail.com --- Hmm, does not seem so (or I am downloading file from the wrong link).
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
--- Comment #6 from Nick Black dank@qemfd.net --- Ugh, apologies, I copied the wrong version over. I've updated both the specfile and the derived SRPM.
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
--- Comment #7 from Nick Black dank@qemfd.net --- I've also updated the specfile and SRPM on #1907017 to include %license.
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
Robert-André Mauchin 🐧 zebob.m@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |POST CC| |zebob.m@gmail.com Assignee|nobody@fedoraproject.org |zebob.m@gmail.com Flags| |fedora-review+
--- Comment #8 from Robert-André Mauchin 🐧 zebob.m@gmail.com --- - License ok - Latest version packaged - Builds in mock - No rpmlint errors - Conforms to Packaging Guidelines
Package approved.
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
--- Comment #9 from Nick Black dank@qemfd.net --- Thanks! https://pagure.io/releng/fedora-scm-requests/issue/31962 requests the git module.
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
--- Comment #10 from Mohan Boddu mboddu@bhujji.com --- (fedscm-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/rust-cty
https://bugzilla.redhat.com/show_bug.cgi?id=1907014
Nick Black dank@qemfd.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|POST |RELEASE_PENDING
package-review@lists.fedoraproject.org