https://bugzilla.redhat.com/show_bug.cgi?id=1577011
Mattia Verga changed:
What|Removed |Added
Status|ASSIGNED|NEW
Assignee|ma...@mavit.org.u
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
Ian McInerney changed:
What|Removed |Added
CC||ian.s.mciner...@ieee.org
--- Comment
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
--- Comment #10 from Peter Oliver ---
It turns out that it's possible to explicitly request sponsorship by raising an
issue at https://pagure.io/packager-sponsors/.
--
You are receiving this mail because:
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
Peter Oliver changed:
What|Removed |Added
Status|NEW |ASSIGNED
--
You are receiving this ma
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
--- Comment #9 from Vaughan ---
(In reply to Peter Oliver from comment #8)
> Best exclude aarch64 until it can be tested. Apart from that, I approve the
> review. Thanks for submitting the package!
>
Done.
https://github.com/agrez/wiringpi/
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
Peter Oliver changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #8 from Pet
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
--- Comment #7 from Vaughan ---
Ok, I have refactored the spec and fixed a few more rpmlint grumbles, plus
added some pkg-config files.
Updated Spec URL:
https://raw.githubusercontent.com/agrez/wiringpi/master/wiringpi.spec
Updated SRPM URL
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
--- Comment #6 from Vaughan ---
(In reply to Peter Oliver from comment #5)
>
> There's a macro, %ldconfig_scriptlets, that will do this for you. See
> https://fedoraproject.org/wiki/Packaging:Scriptlets#Shared_Libraries
>
I didn't know about
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
--- Comment #5 from Peter Oliver ---
(In reply to Vaughan from comment #4)
> > - ldconfig called in %post and %postun if required.
> > Note: /sbin/ldconfig not called in wiringpi-libs
> > See: http://fedoraproject.org/wiki/Packaging/Guide
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
--- Comment #4 from Vaughan ---
Hi Peter, thanks for the review. Please see below for commits / comments.
(In reply to Peter Oliver from comment #3)
> Issues:
> ===
> - Dist tag is present. Use %{?dist} instead of %{dist}. See
> https:/
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
--- Comment #3 from Peter Oliver ---
Thanks for submitting this. Here’s my review:
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Dist t
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
Peter Oliver changed:
What|Removed |Added
Assignee|nob...@fedoraproject.org|ma...@mavit.org.uk
Flags|
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
Peter Oliver changed:
What|Removed |Added
CC||ma...@mavit.org.uk
--- Comment #2 from
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
Vaughan changed:
What|Removed |Added
Blocks||1573194
Referenced Bugs:
https://bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
Vaughan changed:
What|Removed |Added
Blocks||177841 (FE-NEEDSPONSOR)
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=1577011
--- Comment #1 from Vaughan ---
koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=26858050
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this produc
16 matches
Mail list logo