https://bugzilla.redhat.com/show_bug.cgi?id=991221
Linqing Lu changed:
What|Removed |Added
CC||l...@redhat.com
Alias|numatop
https://bugzilla.redhat.com/show_bug.cgi?id=991221
Hanns-Joachim Uhl changed:
What|Removed |Added
Blocks||1496867
--
You are receiving this
https://bugzilla.redhat.com/show_bug.cgi?id=991221
Fedora Update System changed:
What|Removed |Added
Fixed In Version|numatop-1.0.1-5.fc19|numatop-1.0.1-5.fc20
--- Commen
https://bugzilla.redhat.com/show_bug.cgi?id=991221
--- Comment #35 from Dridi Boukelmoune ---
@Peter
I'm not sure to understand the epel6 guidelines, especially around the release
number [1]. I have uploaded a spec [2] that could have worked for both fedora
and epel but starting with f20 rpmd
https://bugzilla.redhat.com/show_bug.cgi?id=991221
--- Comment #36 from Peter Robinson ---
(In reply to Dridi Boukelmoune from comment #35)
> @Peter
>
> I'm not sure to understand the epel6 guidelines, especially around the
> release number [1]. I have uploaded a spec [2] that could have work
https://bugzilla.redhat.com/show_bug.cgi?id=991221
Dridi Boukelmoune changed:
What|Removed |Added
Blocks||177841 (FE-NEEDSPONSOR)
https://bugzilla.redhat.com/show_bug.cgi?id=991221
Zbigniew Jędrzejewski-Szmek changed:
What|Removed |Added
CC||zbys...@in.waw.pl
--- Co
https://bugzilla.redhat.com/show_bug.cgi?id=991221
--- Comment #2 from Zbigniew Jędrzejewski-Szmek ---
Forgot to mention rpmlint status:
W: invalid-license Intel → should be fixed when the license is corrected to BSD
W: spelling-error %description -l en_US runtime → runtime is OK, imho
--
You
https://bugzilla.redhat.com/show_bug.cgi?id=991221
--- Comment #3 from Dridi Boukelmoune ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #1)
> Informal review (I'm not a packager):
>
> Looks like a useful tool!
>
> The license is "BSD" in Fedora [1] (aka "New BSD License", "BSD-new"
https://bugzilla.redhat.com/show_bug.cgi?id=991221
--- Comment #4 from Zbigniew Jędrzejewski-Szmek ---
(In reply to Dridi Boukelmoune from comment #3)
> I was wondering if the spec should explicitely exclude non-intel
> architectures:
> https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Pac
https://bugzilla.redhat.com/show_bug.cgi?id=991221
--- Comment #5 from Michael Schwendt ---
> The package is correct and follows all packaging guidelines, afaict.
Not yet.
> Requires: numactl ncurses
https://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires
> + make -j5
https://bugzilla.redhat.com/show_bug.cgi?id=991221
--- Comment #6 from Zbigniew Jędrzejewski-Szmek ---
In the description: you could explicitly mention that it supports Intel
processors only. From the manpage "numatop supports the Intel Xeon processors:
5500-series, 6500/7500-series, 5600 ser
https://bugzilla.redhat.com/show_bug.cgi?id=991221
Zbigniew Jędrzejewski-Szmek changed:
What|Removed |Added
CC||dridi.boukelmo...@gmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=991221
--- Comment #7 from Dridi Boukelmoune ---
Hi everyone,
Thank you for the review and sorry for the long silence. I have been sponsored
for another package review, so I don't know whether this package still needs
the FE-NEEDSPONSOR dep. Anyway,
https://bugzilla.redhat.com/show_bug.cgi?id=991221
Zbigniew Jędrzejewski-Szmek changed:
What|Removed |Added
CC|package-review@lists.fedora |
|projec
15 matches
Mail list logo
