Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: cpulimit - CPU Usage Limiter for Linux
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Summary: Review Request: cpulimit - CPU Usage Limiter for Linux Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: ashay.humane@gmail.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora
Spec URL: http://ashay.info/rpm/cpulimit.spec SRPM URL: http://ashay.info/rpm/cpulimit-1.1-1.fc11.src.rpm Description:
cpulimit is a simple program that attempts to limit the cpu usage of a process (expressed in percentage, not in cpu time). This is useful to control batch jobs, when you don't want them to eat too much cpu.
It does not act on the nice value or other scheduling priority stuff, but on the real cpu usage. Also, it is able to adapt itself to the overall system load, dynamically and quickly.
More info here: http://cpulimit.sourceforge.net/
The original source did not use autotools and had a simple Makefile. I converted the source to use autotools.
Thank you
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |fabian@bernewireless.net
--- Comment #1 from Fabian Affolter fabian@bernewireless.net 2009-07-24 03:24:44 EDT --- Just some quick comments on your spec file:
You wrote in your sec file... ----%>------ #The source on sourceforge does not use autotools. Source1 does Source0: http://downloads.sourceforge.net/project/cpulimit/cpulimit/cpulimit/cpulimit... Source1: http://ashay.info/rpm/cpulimit-1.1.tar.gz ----<%------
Have you make a patch for the usage of autotools? If yes, send this patch upstream and apply the patch in the spec file.
https://fedoraproject.org/wiki/How_to_create_an_RPM_package#.25prep_section:...
There are more details available about 'Source0'
https://fedoraproject.org/wiki/How_to_create_an_RPM_package#Spec_file_pieces... look for 'Source0'
- 'Source0' seems to be wrong, shouldn't it be 'http://downloads.sourceforge.net/cpulimit/%%7Bname%7D-%%7Bversion%7D.tar.gz' ?
- For valid groups you can use 'less /usr/share/doc/rpm-*/GROUPS'
- %attr (-,root,root) can be left away, it sufficient only to use %{_mandir}/*/*
- Isn't rpmlint complaining about the line length of the %description?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #2 from Ashay ashay.humane@gmail.com 2009-07-24 12:53:23 EDT --- *** Bug 513663 has been marked as a duplicate of this bug. ***
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Itamar Reis Peixoto itamar@ispbrasil.com.br changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |itamar@ispbrasil.com.br
--- Comment #3 from Itamar Reis Peixoto itamar@ispbrasil.com.br 2009-07-24 13:13:02 EDT --- replace these 2 lines
mkdir -p $RPM_BUILD_ROOT%{_mandir}/man1 install -m 644 man/cpulimit.1 $RPM_BUILD_ROOT%{_mandir}/man1/cpulimit.1
with install -Dp -m (only one line do the job)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #4 from Itamar Reis Peixoto itamar@ispbrasil.com.br 2009-07-24 13:16:08 EDT --- is this your first package ?
what's your fedora account (FAS) username ?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #5 from Fabian Affolter fabian@bernewireless.net 2009-07-24 14:21:09 EDT --- (In reply to comment #1)
- Isn't rpmlint complaining about the line length of the %description?
rpmlint output
[fab@laptop09 SRPMS]$ rpmlint cpulimit-1.1-1.fc11.src.rpm cpulimit.src: E: description-line-too-long cpulimit is a simple program that attempts to limit the cpu usage of a process (expressed in percentage, not in cpu time). cpulimit.src: E: description-line-too-long This is useful to control batch jobs, when you don't want them to eat too much cpu. cpulimit.src: E: description-line-too-long It does not act on the nice value or other scheduling priority stuff, but on the real cpu usage. cpulimit.src: E: description-line-too-long Also, it is able to adapt itself to the overall system load, dynamically and quickly. cpulimit.src: W: non-standard-group System Tools cpulimit.src: W: invalid-license GPL 1 packages and 0 specfiles checked; 4 errors, 2 warnings.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |fabian@bernewireless.net Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #6 from Ashay ashay.humane@gmail.com 2009-07-24 18:07:18 EDT --- Thank you for your comments...
I have updated the spec file as suggested. The "convert to autotools" portion is now a patch, which I also sent upstream.
rpmlint errors/warnings are fixed.
Yes, this is my first package. My FAS account name is ashayh.
Thank you
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Itamar Reis Peixoto itamar@ispbrasil.com.br changed:
What |Removed |Added ---------------------------------------------------------------------------- Depends on| |177841(FE-NEEDSPONSOR)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #7 from Itamar Reis Peixoto itamar@ispbrasil.com.br 2009-07-24 18:16:22 EDT --- (In reply to comment #6) please submit the new spec and src.rpm file, remember to bump version to avoid confusion.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #8 from Ashay ashay.humane@gmail.com 2009-07-24 18:28:34 EDT --- (In reply to comment #7)
(In reply to comment #6) please submit the new spec and src.rpm file, remember to bump version to avoid confusion.
Here is the new srpm:
http://ashay.info/rpm/cpulimit-1.1-2.fc11.src.rpm http://ashay.info/rpm/cpulimit.spec
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #9 from Fabian Affolter fabian@bernewireless.net 2009-07-24 18:55:33 EDT --- Your patch add several files. The AUTHORS, COPYING, and all other documentation files should go in %doc.
There is a rpmlint warning. This is easy to fix ;-)
[root@laptop09 i586]# rpmlint cpulimit* cpulimit.i586: W: incoherent-version-in-changelog 1.1 ['1.1-2.fc11', '1.1-2'] 2 packages and 0 specfiles checked; 0 errors, 1 warnings.
A blank line between the changelog entry blocks make the whole changelog easier to ready.
One suggestion: Would it make the life of upstream easier if the man page is installed within the Makefile?
The time stamps are not preserved (https://fedoraproject.org/wiki/Packaging:Guidelines#Timestamps). Almost all files are generated with the patch so this will affect after a new upstream release.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #10 from Fabian Affolter fabian@bernewireless.net 2009-07-24 18:59:07 EDT --- Upstream has a patch (an empty one)
http://sourceforge.net/tracker/?group_id=174425&atid=869186
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #11 from Ashay ashay.humane@gmail.com 2009-07-24 20:36:04 EDT --- Added man file in make. Should I bump the version each and every time I fix something here? (I bumped it)
I have no idea how to fix the time stamps issue...please advise if you do.
http://ashay.info/rpm/cpulimit.spec http://ashay.info/rpm/cpulimit-1.1-3.fc11.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #12 from Fabian Affolter fabian@bernewireless.net 2009-07-25 04:34:00 EDT --- (In reply to comment #11)
Added man file in make.
To the Makefile? Is in this case the line 'install -Dp -m 644 man/cpulimit.1 $RPM_BUILD_ROOT%{_mandir}/man1/cpulimit.1' still needed?
Should I bump the version each and every time I fix something here? (I bumped it)
Everytime you make a change, bump the release. As Itamar already said, this avoid confusion.
I have no idea how to fix the time stamps issue...please advise if you do.
You can use 'make install INSTDIR=$RPM_BUILD_ROOT INSTALL="install -p"' instead of 'make install DESTDIR=$RPM_BUILD_ROOT '. You can change this later when everything is upstream but often it don't harm when it's added before it takes effect.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #13 from Ashay ashay.humane@gmail.com 2009-07-25 15:13:32 EDT --- I removed the "install -Dp -m 644 man/cpulimit.1 $RPM_BUILD_ROOT%{_mandir}/man1/cpulimit.1" " line.
' make install INSTDIR=$RPM_BUILD_ROOT INSTALL="install -p" ' does not work. ( configure and make do not recognize any variable called INSTDIR, and rpmbiuld fails because of that. )
So I added "install -p" to ./configure. Will this work as expected to preserve timestamps?
http://ashay.info/rpm/cpulimit.spec http://ashay.info/rpm/cpulimit-1.1-4.fc11.src.rpm
Thanks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |177841(FE-NEEDSPONSOR) Depends on|177841(FE-NEEDSPONSOR) |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #14 from Fabian Affolter fabian@bernewireless.net 2009-07-25 17:31:56 EDT --- Sorry, my fault... INSTDIR should be DESTDIR.
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"
https://fedoraproject.org/wiki/How_to_create_an_RPM_package#.25install_secti...
As I already said as long the doc files and the man page aren't upstream this is only cosmetically.
(In reply to comment #13)
So I added "install -p" to ./configure. Will this work as expected to preserve timestamps?
No, I was misleading you. Can you please fix this? Then I will make a full review.
BTW, I'm not a sponsor. You can get some hints on that process at the following page https://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored . The long story short: Make some informal reviews of other packages and make some more packages to show that you understand the guidelines and are familiar with them.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #15 from Ashay ashay.humane@gmail.com 2009-07-25 18:46:27 EDT --- Thanks, I will follow the sponsorship process.
Fixed files here:
http://ashay.info/rpm/cpulimit.spec http://ashay.info/rpm/cpulimit-1.1-5.fc11.src.rpm
Any issues now?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Jussi Lehtola jussi.lehtola@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jussi.lehtola@iki.fi
--- Comment #16 from Jussi Lehtola jussi.lehtola@iki.fi 2009-07-29 05:16:17 EDT --- Don't bother with autotools. There's nothing wrong with a package that builds without them, especially when the build process is this simple.
In fact, you don't even need to use the makefile (since it doesn't support optflags). I recommend that you drop Patch0 and
%setup -q
%build gcc $RPM_OPT_FLAGS -lrt -o cpulimit cpulimit.c
%install rm -rf $RPM_BUILD_ROOT install -p -m 755 cpulimit $RPM_BUILD_ROOT/%{_bindir}/cpulimit
**
Besides, your Patch is *creating* the files AUTHORS COPYING README INSTALL ChangeLog
If you want to ship these, list them as extra Source lines. Where did you get these, anyway..? The tarball only contains the source code file and the simple Makefile.
**
I am willing to sponsor you, if you show me you know the Fedora guidelines; the most important of which are http://fedoraproject.org/wiki/Packaging/Guidelines http://fedoraproject.org/wiki/Packaging/ReviewGuidelines http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
To do this you need to make another submission, and perform two informal reviews of packages of other people. Please review only packages not marked with the FE-NEEDSPONSOR blocker as I will have to do the full formal review after you to check that you have got everything right.
After being sponsored, you are able to do reviews of your own.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #17 from Jussi Lehtola jussi.lehtola@iki.fi 2009-07-29 05:18:53 EDT --- (In reply to comment #16)
install -p -m 755 cpulimit $RPM_BUILD_ROOT/%{_bindir}/cpulimit
.. should be, of course, install -D -p -m 755 (I thought something was missing).
Fabian: as you've already started, you can do the review on this one.
I'll throw in extra comments if necessary :)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #18 from Itamar Reis Peixoto itamar@ispbrasil.com.br 2009-07-29 08:00:06 EDT --- (In reply to comment #16)
Don't bother with autotools. There's nothing wrong with a package that builds without them, especially when the build process is this simple.
I agree, but my recommendation is a small patch to makefile without auto-tools. and sending this small patch to upstream of cpulimit.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #19 from Jussi Lehtola jussi.lehtola@iki.fi 2009-07-29 08:26:24 EDT --- (In reply to comment #18)
(In reply to comment #16)
Don't bother with autotools. There's nothing wrong with a package that builds without them, especially when the build process is this simple.
I agree, but my recommendation is a small patch to makefile without auto-tools. and sending this small patch to upstream of cpulimit.
Yes, that is fine too. Then the makefile should look like
BINDIR=/usr/local/bin MANDIR=/usr/local/man INSTALL="install" CFLAGS="-Wall -O2"
all:: cpulimit
cpulimit: cpulimit.c gcc ${CFLAGS} -o cpulimit cpulimit.c -lrt
clean: rm -f cpulimit
install: ${INSTALL} -D -m 755 cpulimit ${DESTDIR}${BINDIR} ${INSTALL} -D -m 644 cpulimit.1 ${DESTDIR}${MANDIR}/man1/cpulimit.1
By default this will compile with "-Wall -O2" and install to /usr/local/bin. In the spec file one would use
%build make %{?_smp_mflags} CFLAGS="$RPM_OPT_FLAGS" to override the CFLAGS variable in Makefile and
%install make DESTDIR=$RPM_BUILD_ROOT BINDIR=%{_bindir} MANDIR=%{_mandir} INSTALL="install -p" install to override the other variables. One can also do without DESTDIR support, then one just prepends $RPM_BUILD_ROOT to MANDIR and BINDIR.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #20 from Itamar Reis Peixoto itamar@ispbrasil.com.br 2009-07-29 08:38:30 EDT --- (In reply to comment #19) the auto-tools previous patch was very intrusive.
patching makefile sounds perfect for me;
you could include CC, something like this.
BINDIR=/usr/local/bin MANDIR=/usr/local/man INSTALL="install" CFLAGS="-Wall -O2"
CC="gcc"
all:: cpulimit
cpulimit: cpulimit.c ${CC} ${CFLAGS} -o cpulimit cpulimit.c -lrt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Juha Tuomala tuju@iki.fi changed:
What |Removed |Added ---------------------------------------------------------------------------- URL| |http://cpulimit.sourceforge | |.net/ CC| |tuju@iki.fi Alias| |cpulimit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #21 from Fabian Affolter fabian@bernewireless.net 2009-07-29 19:38:34 EDT --- (In reply to comment #17)
Fabian: as you've already started, you can do the review on this one.
I'll throw in extra comments if necessary :)
Ok, I will. It good to know are you are watching over my shoulder ;-)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #22 from Ashay ashay.humane@gmail.com 2009-07-29 20:14:28 EDT --- Ok, so as suggested, I got rid of autotools and made a patch for the Makefile.
The man page was not included by upstream, I got that from the debian package. If I do not include the man page, rpmlint warns about no documentation. So I decided to include the man page as Source1. (and sent it upstream on sf.net)
If that's not a good idea, I will remove the man page and let the warning be.
Thanks everyone for their comments.
Jussi, thanks for the sponsorship offer. I'll make a few more packages and review some packages soon. I intend to make/maintain packages on a regular basis.
http://ashay.info/rpm/cpulimit.spec http://ashay.info/rpm/cpulimit-1.1-6.fc11.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #23 from Jussi Lehtola jussi.lehtola@iki.fi 2009-07-29 20:26:43 EDT --- (In reply to comment #22)
Ok, so as suggested, I got rid of autotools and made a patch for the Makefile.
The man page was not included by upstream, I got that from the debian package. If I do not include the man page, rpmlint warns about no documentation. So I decided to include the man page as Source1. (and sent it upstream on sf.net)
If that's not a good idea, I will remove the man page and let the warning be.
OK. Debian considers not having a man file a packaging error, so they have the tendency of generating them if they do not already exist.
In Fedora we do not do so; we try to stay as close to upstream as possible and thus if there is no documentation in the tarball we don't ship any, either. I wouldn't ship the man page (generally there's no guarantee that it isn't obsolete), but you should send it upstream to see if they could include it in the next release. Also ask them to add the relevant license file, i.e. http://www.gnu.org/licenses/gpl-2.0.txt
Some times you don't have to care about rpmlint warnings. The no-doc warning is the most common of these cases; you just have to check if there is something that should be put there.
Thanks everyone for their comments.
Jussi, thanks for the sponsorship offer. I'll make a few more packages and review some packages soon. I intend to make/maintain packages on a regular basis.
Good, since packaging shouldn't be a one-off activity :)
Paste the numbers of the BZ entries here so I can check them out.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #24 from Ashay ashay.humane@gmail.com 2009-07-29 20:46:46 EDT --- Ok, I removed the man page. Here are the new files:
http://ashay.info/rpm/cpulimit.spec http://ashay.info/rpm/cpulimit-1.1-7.fc11.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #25 from Fabian Affolter fabian@bernewireless.net 2009-08-02 09:23:58 EDT --- After the rebuild of the source RPM rpmlint says:
[fab@laptop09 i586]$ rpmlint cpulimit* cpulimit.i586: W: no-documentation cpulimit-debuginfo.i586: E: debuginfo-without-sources 2 packages and 0 specfiles checked; 1 errors, 1 warnings.
For it seems that the compiler flags aren't honored. Comment #19 said what to do ;-)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #26 from Jussi Lehtola jussi.lehtola@iki.fi 2009-08-02 09:53:43 EDT --- (In reply to comment #25)
After the rebuild of the source RPM rpmlint says:
[fab@laptop09 i586]$ rpmlint cpulimit* cpulimit.i586: W: no-documentation cpulimit-debuginfo.i586: E: debuginfo-without-sources 2 packages and 0 specfiles checked; 1 errors, 1 warnings.
For it seems that the compiler flags aren't honored. Comment #19 said what to do ;-)
i.e. you need make %{?_smp_mflags} CFLAGS="$RPM_OPT_FLAGS" instead of make %{?_smp_mflags}
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #27 from Ashay ashay.humane@gmail.com 2009-08-03 11:32:48 EDT --- Ok, I added that:
http://ashay.info/rpm/cpulimit.spec http://ashay.info/rpm/cpulimit-1.1-8.fc11.src.rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #28 from Jussi Lehtola jussi.lehtola@iki.fi 2009-08-06 07:47:17 EDT --- Have you made any other submissions of performed informal reviews?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |XiaShing@gmail.com
--- Comment #29 from Fabian Affolter fabian@bernewireless.net 2009-09-17 14:05:46 EDT --- *** Bug 524006 has been marked as a duplicate of this bug. ***
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #30 from Xia Shing Zee XiaShing@gmail.com 2009-09-17 20:51:16 EDT --- Is this review request still ongoing?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(ashay.humane@gmai | |l.com)
--- Comment #31 from Fabian Affolter fabian@bernewireless.net 2009-09-29 03:44:55 EDT --- Ashay, did you make some informal reviews or other packages?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
--- Comment #32 from Fabian Affolter fabian@bernewireless.net 2009-10-23 09:58:40 EDT --- ping?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Ashay Humane ashay.humane@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(ashay.humane@gmai | |l.com) |
--- Comment #33 from Ashay Humane ashay.humane@gmail.com 2009-10-23 11:32:36 EDT --- Hi Fabian
Sorry for the delay. I've been on vacation for a long time with no practical access to computers/internet.
I've done one informal review so far. And created another package (adtool)
I'll be back in a few weeks and continue...
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |NEW AssignedTo|fabian@bernewireless.net |nobody@fedoraproject.org
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Fabian Affolter fabian@bernewireless.net changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(ashay.humane@gmai | |l.com)
--- Comment #34 from Fabian Affolter fabian@bernewireless.net 2010-01-04 05:09:04 EDT --- Ashay, is there any progress? There was another review request by Xia submitted #524006, if you have no time it would be an option to close this review and go with the one of Xia. What do you think?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Ashay Humane ashay.humane@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(ashay.humane@gmai | |l.com) |
--- Comment #35 from Ashay Humane ashay.humane@gmail.com 2010-01-04 12:34:20 EDT --- Please close this and go ahead with Xias review. Thanks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Jason Tibbitts tibbs@math.uh.edu changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Blocks|177841(FE-NEEDSPONSOR) |201449(FE-DEADREVIEW) Resolution| |NOTABUG
--- Comment #36 from Jason Tibbitts tibbs@math.uh.edu 2010-01-21 00:13:03 EST --- Closing as requested.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Thomas Spura tomspur@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |tomspur@fedoraproject.org Resolution|NOTABUG |DUPLICATE
--- Comment #37 from Thomas Spura tomspur@fedoraproject.org 2012-01-11 17:02:41 EST ---
*** This bug has been marked as a duplicate of bug 772406 ***
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=513541
Thomas Spura tomspur@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|201449(FE-DEADREVIEW) |
package-review@lists.fedoraproject.org