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

[Fedora Electronic Lab]

#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 zap):

 Okay, here goes another iteration of the spec file :)

 Replying to [comment:8 chitlesh]:
 > #1: warning: deprecated conversion from string constant to ‘char*’[[BR]]
 > can you patch this ?
 done.

 > #2: fix the desktop-file-install, see sherry's spec so that the desktop
 file gets into the Electronic section.
 my older spec included a direct patch for .desktop file to place it in the
 Electronics section, but indeed it's better to do this at %install time.

 > #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
 entry.
 Ah, I was looking at the wrong .spec file and didn't see that. Fixed.

 > #4: same, add empty lines to separate different blocks of the spec.
 Hmm, my .spec files always separates blocks from each other with a empty
 line. I don't see what I can fix here?

 As always, latest .src.rpm here:
 http://cs.ozerki.net/zap/pub/rpm/qtdmm-0.8.12-1.fc9.zap.src.rpm

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