[Bug 223724] Review Request: fvwm - window manager

bugzilla at redhat.com bugzilla at redhat.com
Wed Jan 24 01:26:26 UTC 2007


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: fvwm - window manager


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


pertusus at free.fr changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pertusus at free.fr




------- Additional Comments From pertusus at free.fr  2007-01-23 20:26 EST -------
* regarding the license, I have spotted some files without license
  but with a mention to an author or even a copyright notice. 
  When there is a copyright and no license, the default license
  is a restrictive license (no modification nor redistribution)
  which is non free and GPL incompatible.
  
it is the case for
bin/fvwm-config
bin/fvwm-convert-2.2 (although the comments seem to indicate that
                      it is public domain)
bin/fvwm-convert-2.4

utils/fvwm_make_*.sh  (although these are examples and so are likely
                       in the public domain)

modules/FvwmWharf/ASSound/ASSound.c

* I think that the summary could be shortened, ICCM compliance and
  for the X Window system are not very informative. Maybe something along

Summary:        Highly configurable multiple virtual desktop window manager

* fvwm-menu-headlines should use htmlview instead of netscape

* fvwm-bug requires %{_sbindir}/sendmail

* fvwm-menu-xlock requires xlock

* fvwm-menu-directory could use mimeopen -n instead of EDITOR to 
  open files.

* in fvwm/ConfigFvwmSetup the default menu is badly suited for 
  fedora. I propose removing rxvt, replacing the utility submenu 
  a dynamically generated menu using the freedesktop standard.
  For that, I found a python script using pyxdg which could 
  be suitable:
http://www.cl.cam.ac.uk/~pz215/fvwm-scripts/scripts/xdg-menu.py
  However, I found 2 issues with that script, when calling it with
  xdg-menu.py /etc/xdg/menus/applications.menu > somefile
1. the accented characters make it break
2. the top level menu hasn't a reproducible name which could be handy
   to reference it in a Popup entry. 
  I reported those issues upstream, with a patch to add a -m option
  to specify the toplevel menu name

Then AddToMenu MenuFvwmUtilities  could become something along:

PipeRead "fvwm-xdg-menu-top -m MenuFvwmUtilities /etc/xdg/menus/applications.menu"


* there is a missing
Requires: xterm

-- 
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 Fedora-package-review mailing list