[Fedora-electronic-lab] Re: [Fedora Electronic Lab] #23: Package qtdmm

Fedora Electronic Lab trac at fedorahosted.org
Wed Jun 10 09:11:07 UTC 2009

#23: Package qtdmm
  Reporter:  zap            |       Owner:  sherry151
      Type:  task           |      Status:  new      
  Priority:  minor          |   Milestone:  Fedora 12
 Component:  analog design  |     Version:  devel    
Resolution:                 |    Keywords:           
Comment (by chitlesh):

 Replying to [comment:7 zap]:
 > sherry151, thanks very much for review! Especially for the rpmlint hint,
 this seems a nice tool to avoid childish packaging problems.
 > I have fixed almost all mentioned problems and will upload the spec file
 > Also I've updated the src.rpm here:
 > http://cs.ozerki.net/zap/pub/rpm/qtdmm-0.8.12-1.fc9.zap.src.rpm
 > Only a few objections:
 >  * Still I would like to keep the older changelog to credit the original
 spec file author. This is imho a matter of fairness, but if you insist I
 will remove it.

 I said above they should be kept.

 >  * 'Requires qt3' is needed because it's a bit of difference between
 Requires and Build-Requires.

 #1: warning: deprecated conversion from string constant to ‘char*’[[BR]]
 can you patch this ?

 #2: fix the desktop-file-install, see sherry's spec so that the desktop
 file gets into the Electronic section.

 #3: changelog adopt the same changelog style as sherry did. The version-
 release at the end of the line + an empty line in between two changelog

 #4: same, add empty lines to separate different blocks of the spec.
 Although this is not critical for the package, it makes the spec file hard
 to read without spending too much time. See sherry's spec again or my spec
 file on http://chitlesh.fedorapeople.org/RPMS/

Ticket URL: <https://fedorahosted.org/fedora-electronic-lab/ticket/23#comment:8>
Fedora Electronic Lab <https://fedorahosted.org/fedora-electronic-lab>
Design, Simulate and Program electronics.

More information about the Fedora-electronic-lab-list mailing list