[Bug 211551] Review Request: fluxter - a pager for use in fluxbox
bugzilla at redhat.com
bugzilla at redhat.com
Thu Oct 19 22:39:35 UTC 2006
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: fluxter - a pager for use in fluxbox
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=211551
pertusus at free.fr changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|Review Request: fluxter - a |Review Request: fluxter - a
|pager for use in fluxbox |pager for use in fluxbox
CC| |pertusus at free.fr
------- Additional Comments From pertusus at free.fr 2006-10-19 18:39 EST -------
* You don't need to set CFLAGS, they are set by %configure.
* if you add --enable-debug to %configure, the -s isn't added
and the debuginfo package is created.
* Why do you change fluxbox to fluxter in Makefiles? The fluxter
files seems to live happily in %{_datadir}/fluxbox. Even if fluxter
is usefull without fluxbox.
* Missing Buildrequires, I spotted libX11-devel and libSM-devel.
* is the Requires fluxbox really needed? Doesn't it work with other
wm?
* No need to have the package name in the summary.
--
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.
More information about the package-review
mailing list