Bug 285571

rpmlint on SRPM gave me
seekwatcher.src: W: no-%build-section
The spec file does not contain a %build section.  Even if some packages
don't directly need it, section markers may be overridden in rpm's
configuration to provide additional "under the hood" functionality, such as
injection of automatic -debuginfo subpackages.  Add the section, even if
empty.


Where can I find mencoder?

package installation failed with error: Failed dependencies:
        mencoder is needed by seekwatcher-0.7-1.fc8.noarch

Just confirmed on #fedora-devel and this package cannot make into Fedora because
of missing package dependency mencoder.

If by any chance you can use any alternate to mencoder then we can think this
for inclusion in Fedora.

Thanks for the review; I'll add a %build.

Sorry about mencoder, that was a thinko on my part.  seekwatcher can still make
still graphs, just not movies.  So I think we can get it in with limited
functionality.  I'll see about patching it up to remove the option, or something
along those lines.

-Eric

Eric how about either trying to build a modular version and submit the mencoder
dependent part to rpmfusion or maybe submit the whole package to rpmfusion ?

hmm apparently the beta1 theora release has png2theora in theora-tools... pity
F8 doesn't have that release, yet... maybe I can ask for just that to be
included in the current package.  Chris Mason would be willing to use those free
tools rather than the mpg stuff.  I'll take a look.

-Eric

(In reply to comment #3)
> Thanks for the review; I'll add a %build.
ok
> 
> Sorry about mencoder, that was a thinko on my part.  seekwatcher can still make
> still graphs, just not movies.  So I think we can get it in with limited
> functionality.  I'll see about patching it up to remove the option, or something
> along those lines.
> 
  Will be happy to review this then :)

Well, spent too much time on this today :)  png2theora is already in the
libtheora src.rpm we have, just not installed.  I have it making a "movie" but
something is odd; it looks pretty wrong.

-Eric

any progress here?

Nope.  The theora guys are stumped too, but the movie comes out wrong.

The static graph side of things works, though.  It's just the animated movies
that don't have a solution at this point.

-Eric

I may just patch out the movie-making part, or maybe better yet make it dynamic
so it's available *if* mencoder or other suitable app is found.

Sorry for the delay on this, it's a low priority for me even though I'd like to
get it in.

--Eric

very latest seekwatcher repo has a patch which simply makes the movie options
"go away" if mencoder is not found.  When chris makes that a point release, I'll
make a new package for review.

Thanks,
-Eric

See also bug #401681 : Add filtering flags to png_read_png ... the (now) very
latest seekwatcher can use png2theora, and with the fix in the aforementioned
bug, it works.  So, I'll try to wrap this one up soon.

Thanks,
-Eric

Ok.  Sorry for the delay... let's try this:

http://people.redhat.com/esandeen/seekwatcher-rpm/seekwatcher-0.8-1.fc8.src.rpm
http://people.redhat.com/esandeen/seekwatcher-rpm/seekwatcher.spec

Thanks!

-Eric

looks missing python-devel
In build.log I got
/usr/lib/rpm/pythondeps.sh: line 8: python: command not found

argh, mock produced the rpm so didn't check the logs.

... fix fix fix ...

Added python to reqs/buildreqs

http://people.redhat.com/esandeen/seekwatcher-rpm/seekwatcher-0.8-2.fc8.src.rpm

No complaints from mock this time.

Thanks,
-Eric

And keeping up with chris....

http://people.redhat.com/esandeen/seekwatcher-rpm/seekwatcher-0.9-1.fc8.src.rpm

(In reply to comment #17)
> And keeping up with chris....
> 
> http://people.redhat.com/esandeen/seekwatcher-rpm/seekwatcher-0.9-1.fc8.src.rpm

with this SRPM I got,

seekwatcher.noarch: I: checking
seekwatcher.noarch: W: incoherent-version-in-changelog 0.9 0.9-1.fc8
The last entry in %changelog contains a version identifier that is not
coherent with the epoch:version-release tuple of the package.

seekwatcher.src: I: checking
seekwatcher.src: W: no-%build-section
The spec file does not contain a %build section.  Even if some packages
don't directly need it, section markers may be overridden in rpm's
configuration to provide additional "under the hood" functionality, such as
injection of automatic -debuginfo subpackages.  Add the section, even if
empty.

Also,
  make sure to keep timestamps
http://fedoraproject.org/wiki/Packaging/Guidelines#head-0239576e441f9ef53d175c4aec8c12868dffb5ab

And if you like you can use defattr as
defattr(-,root,root,-)

*sigh* I guess maybe I'm just really really bad at this ;-)

Is that rpmlint output?  Ok, now my build log looks clean and rpmlint too.

Ok... once more with feeling:

http://people.redhat.com/esandeen/seekwatcher-rpm/seekwatcher-0.9-2.fc8.src.rpm

Thanks for puttting up with me :)  Hope I've got it now.

-Eric

Thanks for updates.

Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
e4c554577e9298fc70fd162bdc647746  seekwatcher-0.9.tar.bz2
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ BuildRequires are proper.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ no .pc file present.
+ -devel subpackage does exist.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Not a GUI App.
APPROVED.

Thanks for the review(s)!

Can you add a template here with what branches you want and such?
See: 
http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure

Reset the fedora-cvs flag when you are ready. 

Sorry 'bout that:

New Package CVS Request
=======================
Package Name: seekwatcher
Short Description: Utility for visualizing block layer IO patterns and performance
Owners: sandeen
Branches: F-8
InitialCC:
Cvsextras Commits: yes

Thanks,
-Eric

Imported & built... thanks everyone!

Package Change Request
======================
Package Name: seekwatcher
New Branches: EL-5

cvs done.