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@xxxxxxxxxx changed: What |Removed |Added ---------------------------------------------------------------------------- Severity|normal |medium ------- Additional Comments From green@xxxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review