[Bug 190029] Review Request: whysynth-dssi - DSSI software synthesizer plugin

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: whysynth-dssi - DSSI software synthesizer plugin


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190029


bugzilla at redhat.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |medium




------- Additional Comments From green at redhat.com  2007-03-27 05:38 EST -------
(In reply to comment #18)
> (In reply to comment #16)
> > (In reply to comment #15)
> > > E: whysynth-dssi non-standard-dir-perm
/usr/share/doc/whysynth-dssi-20060122/doc
> > > 02755 (this should be fixed)
> > 
> > I don't see this now.
> 
> Hm, this problem only occurs with a rpmbuilt package, but not with the mock build.

I can't reproduce this problem. 

> 
> > Why do you install the icon twice? I think the icon below pixmaps can / should
> > be skipped.
> 
> > Ok.
> 
> Eh, please pardon my bad english, I meant keep the icon that is installed in
> %{_datadir}/icons/hicolor and do not install it in 
> $RPM_BUILD_ROOT%{_datadir}/pixmaps/, so exactly the other way round.
> 
> %{_datadir}/icons/hicolor

Ok, fixed.

New bits here:
http://people.redhat.com/green/FE/devel/whysynth-dssi-20060122-9.src.rpm
http://people.redhat.com/green/FE/devel/whysynth-dssi.spec

Is this package OK now?

Thanks!




-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.