[Bug 651123] Review Request: sarGraphs - Graphical System Usage Web Interface

bugzilla at redhat.com bugzilla at redhat.com
Tue Nov 9 17:08:26 UTC 2010


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=651123

--- Comment #7 from Jeffrey Ness <jness at flip-edesign.com> 2010-11-09 12:08:25 EST ---
(In reply to comment #6)
> (In reply to comment #5)
> > (In reply to comment #2)
> > > Hi Jeffrey
> > > 
> > > your package will not build as it presently stands.
> > > 
> > > Please be sure to install rpmdevtools and follow all the guidelines here to
> > > build your Fedora package:
> > > http://fedoraproject.org/wiki/How_to_create_an_RPM_package
> > > and here
> > > http://fedoraproject.org/wiki/Packaging/Guidelines
> > > 
> > > A few things in particular:
> > > - your LICENSE file in your source states GPLv2 whereas you have specified GPL
> > > in the spec file
> > > - you need to specify the full URL of your Source0
> > > - your release number should start at 1 not 0
> > > - you cannot unpack files into /opt when you are building packages in your spec
> > > file (you should not be installing anything under /opt in any case). The line
> > > 'setup -q' will unpack the source named under Source0 for you into
> > > ~/rpmbuild/BUILD/%{name}-%{version} where your package is built. Everything in
> > > the spec file is executed relative to here. In your case you could omit the
> > > %build section and simply move the required files from this area to your
> > > buildroot in your %install section.
> > > - your %post is better suited to a README file
> > > - you will also need to clean up your files section, including adding a %doc
> > > line to include your license file and any other documents you wish to be
> > > included in the package such as AUTHOR, README - refer to the links above
> > > 
> > > Once you've attacked those things and can successfully build your package using
> > > rpmbuild be sure to run rpmlint against the generated src package and post thre
> > > results back here.
> > > 
> > > hope that helps,
> > > 
> > > Brendan
> > 
> > Hello Brendan,
> > 
> > Thank you very much for this information, it has been very useful in my
> > entering of the RPM development scene. 
> > 
> > I have been re-rewriting my SPEC file and took in your considerations above:
> > 
> > Spec URL: http://flip-edesign.com/source/sarGraphs-1.1-1.spec
> > SRPM URL: http://flip-edesign.com/source/sarGraphs-1.1-1.src.rpm
> > 
> > --
> > 
> > I have also found the 'rpmlint' tool to be a great asset, below you can find
> > the current results:
> > 
> > rpmlint ../SRPMS/sarGraphs-1.1-1.src.rpm 
> > sarGraphs.src: W: summary-not-capitalized sarGraphs takes the output of sysstat
> > and creates a graphical PHP Web interface.
> > sarGraphs.src: W: summary-ended-with-dot sarGraphs takes the output of sysstat
> > and creates a graphical PHP Web interface.
> > sarGraphs.src: W: name-repeated-in-summary sarGraphs
> > sarGraphs.src: E: invalid-spec-name
> > sarGraphs.src:24: E: files-attr-not-set
> > sarGraphs.src:25: E: files-attr-not-set
> > sarGraphs.src: W: no-cleaning-of-buildroot %install
> > sarGraphs.src: W: no-cleaning-of-buildroot %clean
> > sarGraphs.src: W: no-buildroot-tag
> > sarGraphs.src: W: no-%build-section
> > sarGraphs.src: W: no-%clean-section
> > 1 packages and 0 specfiles checked; 3 errors, 8 warnings.
> > --
> > 
> > I believe the 3 errors can be over looked. I do not need to set the default
> > file attributes, and I believe my SPEC file naming convention matches that of
> > http://fedoraproject.org/wiki/Packaging/Guidelines#Naming. As for the warnings
> > nothing stands out to something I would need to correct.
> > 
> > I'm going to attempt to build on the Fedora Build system , however at the
> > moment it seems the site 'https://koji.fedoraproject.org/koji/login' is under
> > high load. I will attempt one of these mock builds when the site is available.
> > 
> > Thanks
> > Jeffrey-
> 
> Greetings,
> 
> I have been able to run this Source RPM on the Build system, unfortunately it
> failed. While checking the build.log I get this message below (failed to write
> to /usr/share):
> 
> + /bin/cp -a /builddir/build/BUILD/sarGraphs-1.1 /usr/share
> /bin/cp: cannot create directory `/usr/share/sarGraphs-1.1': Permission denied
> error: Bad exit status from /var/tmp/rpm-tmp.89564 (%install)
> 
> --
> 
> I guess I'm abit confused at this point, how would I go about moving my source
> from the RPM to its final resting place?
> 
> Thanks for any advise.
> Jeffrey-

Alright, I believe I have resolved my issues and created this RPM the correct
way.

I have been successful on building the SRPM on the Build System:

$ koji build --arch-override=i386 --scratch dist-5E-epel
/usr/src/redhat/SRPMS/sarGraphs-1.1-1.src.rpm 
Uploading srpm: /usr/src/redhat/SRPMS/sarGraphs-1.1-1.src.rpm
[====================================] 100% 00:00:19   4.42 MiB 229.03 KiB/sec
Created task: 2590548
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=2590548
Watching tasks (this may be safely interrupted)...
2590548 build (dist-5E-epel, sarGraphs-1.1-1.src.rpm): free
2590548 build (dist-5E-epel, sarGraphs-1.1-1.src.rpm): free -> open
(ppc05.phx2.fedoraproject.org)
  2590549 buildArch (sarGraphs-1.1-1.src.rpm, i386): open
(x86-02.phx2.fedoraproject.org)
  2590549 buildArch (sarGraphs-1.1-1.src.rpm, i386): open
(x86-02.phx2.fedoraproject.org) -> closed
  0 free  1 open  1 done  0 failed
2590548 build (dist-5E-epel, sarGraphs-1.1-1.src.rpm): open
(ppc05.phx2.fedoraproject.org) -> closed
  0 free  0 open  2 done  0 failed

2590548 build (dist-5E-epel, sarGraphs-1.1-1.src.rpm) completed successfully

--

The results of my rpmlint:

$ rpmlint /usr/src/redhat/SRPMS/sarGraphs-1.1-1.src.rpm 
sarGraphs.src: W: summary-not-capitalized sarGraphs takes the output of sysstat
and creates a graphical PHP Web interface.
sarGraphs.src: W: summary-ended-with-dot sarGraphs takes the output of sysstat
and creates a graphical PHP Web interface.
sarGraphs.src: W: name-repeated-in-summary sarGraphs
sarGraphs.src: E: no-changelogname-tag
sarGraphs.src: E: invalid-spec-name
sarGraphs.src:32: E: files-attr-not-set
sarGraphs.src:33: E: files-attr-not-set
sarGraphs.src: W: no-%build-section
1 packages and 0 specfiles checked; 4 errors, 4 warnings.

--

Jeffrey-

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.



More information about the package-review mailing list