Bug 249590 - Review Request: and - Auto nice daemon
Summary: Review Request: and - Auto nice daemon
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Jason Tibbitts
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-07-25 16:44 UTC by Jochen Schmitt
Modified: 2007-11-30 22:12 UTC (History)
2 users (show)

Fixed In Version: 1.2.2-3.fc7
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-08-06 17:55:37 UTC
Type: ---
Embargoed:
j: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Jochen Schmitt 2007-07-25 16:44:59 UTC
Spec URL: http://www.herr-schmitt.de/pub/and/and.spec
SRPM URL: http://www.herr-schmitt.de/pub/and/and-1.2.2-1.src.rpm
Description:
The auto nice daemon renices and even kills jobs according to their CPU time,
owner, and command name. This is especially useful on production machines with
lots of concurrent CPU-intensive jobs and users that tend to forget to
nice their jobs.

Comment 1 Jochen Schmitt 2007-07-26 14:57:05 UTC
Some Improvements:

Spec URL: http://www.herr-schmitt.de/pub/and/and.spec
SRPM URL: http://www.herr-schmitt.de/pub/and/and-1.2.2-2.src.rpm

Comment 2 Jason Tibbitts 2007-07-29 04:08:37 UTC
For some reason, attempting to fetch the -2 package redirects me to the -1 package:

Connecting to www.herr-schmitt.de|82.165.107.151|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: http://www.herr-schmitt.de/pub/and/and-1.2.2-1.src.rpm [following]


Comment 3 Jochen Schmitt 2007-07-29 17:54:27 UTC
I have uploaded the package again and tested the download on my own machine.

The download should works now.

Comment 4 Jason Tibbitts 2007-07-29 18:51:54 UTC
I'm still getting redirected:

> wget http://www.herr-schmitt.de/pub/and/and-1.2.2-2.src.rpm
--13:49:51--  http://www.herr-schmitt.de/pub/and/and-1.2.2-2.src.rpm
Resolving www.herr-schmitt.de... 82.165.107.151
Connecting to www.herr-schmitt.de|82.165.107.151|:80... connected.
HTTP request sent, awaiting response... 301 Moved Permanently
Location: http://www.herr-schmitt.de/pub/and/and-1.2.2-1.src.rpm [following]
--13:49:52--  http://www.herr-schmitt.de/pub/and/and-1.2.2-1.src.rpm
Connecting to www.herr-schmitt.de|82.165.107.151|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 33569 (33K) [audio/x-pn-realaudio-plugin]
Saving to: `and-1.2.2-1.src.rpm'

100%[=====================================================================================>]
33,569      86.7K/s   in 0.4s

13:49:53 (86.7 KB/s) - `and-1.2.2-1.src.rpm' saved [33569/33569]

Firefox gets redirected as well.

Comment 5 Jochen Schmitt 2007-07-29 19:06:12 UTC
OK. I have delete and recreate the and directory.

Now, you may get the 301 respone from the server, but the right package.



Comment 6 Jason Tibbitts 2007-07-29 19:57:35 UTC
Nope:

> wget http://www.herr-schmitt.de/pub/and/and-1.2.2-2.src.rpm
--14:53:28--  http://www.herr-schmitt.de/pub/and/and-1.2.2-2.src.rpm
Resolving www.herr-schmitt.de... 82.165.107.151
Connecting to www.herr-schmitt.de|82.165.107.151|:80... ^[[6~connected.
HTTP request sent, awaiting response... 300 Multiple Choices
Length: unspecified [text/html]
Saving to: `and-1.2.2-2.src.rpm'

    [ <=>                                                                      
           ] 475         --.-K/s   in 0s

14:53:28 (36.9 MB/s) - `and-1.2.2-2.src.rpm' saved [475]

The 475 bytes I get are:

<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<HTML><HEAD>
<TITLE>300 Multiple Choices</TITLE>
</HEAD><BODY>
<H1>Multiple Choices</H1>
The document name you requested (<code>/pub/and/and-1.2.2-2.src.rpm</code>)
could not be found on this server.
However, we found documents with names similar to the one you
requested.<p>Available documents:
<ul>
<li><a
href="/pub/and/and-1.2.2-2.fc7.src.rpm">/pub/and/and-1.2.2-2.fc7.src.rpm</a>
(common basename)
</ul>
</BODY></HTML>

Oh, the links aren't correct.  It really shouldn't be this difficult to put up a
couple of files for downloading.  I think I have managed to fetch the proper
file now.

Comment 7 Jochen Schmitt 2007-07-30 15:07:19 UTC
At first, I'm very sorry for any inconvenience caused by this technical issue.

I have found out, that my ISP has installed a mechanism which try to guess the
rigth name of the document on the webserver if you type in a wrong URL.

The following URL should works properly:

Spec URL: http://www.herr-schmitt.de/pub/and/and.spec
SRPM URL: http://www.herr-schmitt.de/pub/and/and-1.2.2-2.fc7.src.rpm

Comment 8 Jason Tibbitts 2007-07-30 18:15:34 UTC
I think something is wrong with your scriptlet dependencies; post does not call
/sbin/service.  I think perhaps you meant
  Requires(postun):  /sbin/service

Since this is just a two-character typo and is the only thing I see wrong with
this package, I'll go ahead and approve it and you can fix it when you check in.

Review:
* source files match upstream:
   d34a09d7a396bcf19beb0002195c5862b62292b04fd1fb5a83ae382955f28118
   and-1.2.2.tar.gz
* package meets naming and versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
* license text included in package.
* latest version is being packaged.
* BuildRequires are proper (none)
* compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, x86_64).
* package installs properly.
* debuginfo package looks complete.
* rpmlint is silent.
* final provides and requires are sane:
   config(and) = 1.2.2-2.fc8
   and = 1.2.2-2.fc8
  =
   /bin/sh
   /sbin/chkconfig
   /sbin/service
   config(and) = 1.2.2-2.fc8
* %check is not present; no test suite upstream.  I installed the package and 
   made sure the service would start and stop properly.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
X scriptlets look OK but dependencies are slightly off.
* init script looks good and seems to conform to LSB requirements.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

APPROVED, just fix the %postun scriptlet dependency.

Comment 9 Jochen Schmitt 2007-07-30 18:35:01 UTC
New Package CVS Request
=======================
Package Name: and
Short Description: Auto nice daemon
Owners: Jochen
Branches: F-7
InitialCC: 



Comment 10 Jason Tibbitts 2007-07-30 18:39:12 UTC
CVS done.

Comment 11 Fedora Update System 2007-07-31 22:51:20 UTC
and-1.2.2-3.fc7 has been pushed to the Fedora 7 testing repository.  If problems still persist, please make note of it in this bug report.

Comment 12 Fedora Update System 2007-08-06 17:55:34 UTC
and-1.2.2-3.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.