[Bug 463211] Review Request: notify-sharp - A C# implementation for Desktop Notifications

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


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





--- Comment #14 from Paul F. Johnson <paul@xxxxxxxxxxxxxxxxxxxxxx>  2008-10-09 17:59:00 EDT ---
#12 -

The docs should be in their own subpackage. It doesn't matter if there is only
one file in the -devel subpackage, the guidelines say that the .pc file should
be in the -devel subpackage, so there it goes. Quite a few mono -devel
subpackages only contain the .pc file (mono-basic generates it's own from the
spec file)

The patch is also not really required

sed -i -e 's!${exec_prefix}/lib!%{_libdir}!' notify-sharp.pc.in

should fix the problem with minimal fuss

#13 -

You need to remove the smp_mflags from the build script (it should fix the
problem). Again, mono apps don't always like smp_mflags being set (monodevelop
really gets upset at parallel builds going on).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]