James,
I use LXDE and I would like give cwm a go. If I can get it working on
my system I'll certainly work with you to get it uploaded into Debian. I
would not really call myself a competent X11 administrator so maybe I am
not your ideal uploader. On the other hand maybe I'm ideal because I
will ask all the right stupid questions. And because they're stupid
questions please feel free to tell me why they are stupid questions.
So here are some stupid questions for starters:
1,) I'm running LXDE and openbox and I know nothing about X11. How do I
get cwm working? Could I have that in a README.Debian file please?
2.) Why doesn't the package have a "Provides: x-window-manager" clause
like openbox does?
3.) What steps have you taken to check that cwm will fit into the Debian
environment? Have you looked at say
http://wiki.debian.org/WindowManager? Compared with openbox?
Then there are issues I picked up on myself but found reiterated in the
existing bug report:
4.) "I would remove the last sentence of the first
paragraph though (about the code that used to come from 9wm), as it
doesn't seem very relevant anymore."
More generally I feel your long description should answer the following
questions: What is cwm?; Why might I want to use cwm? Why might I not
want want to use cwm?
I think you're almost there but as it stands the bits about .cwmrc and
virtual desktops seem out of place. You may want that information in
there but I would suggest thinking about it again.
5.) "And if you don't use a VCS for your packaging, you should remove
those commented-out lines."
Have you considered using collab-maint as a repository?
http://wiki.debian.org/Alioth/PackagingProject?highlight=%28CategoryAlioth%29
6.) "There is no upstream changelog as there is none
provided." "The README doesn't contain useful information for end-users,
so you shouldn't install it."
Actually the README contains the upstream changelog. So you should
install the README as the upstream changelog. [More strategically you
could ask upstream to generate a report from the Openbsd repository to
get an upstream changelog. Or you could do that yourself perhaps.]
For completeness I'll also reiterate the comments from Benoit Knecht:
7.) In debian/control, the debhelper version dependency should simply be
">= 9" instead of ">= 9.0.0".
8.) And in the same file, the long description contains a few
double-spaces.
9.) "But prehaps you should consider Depending on xserver-xorg (or at
least Recommend it, if that makes more sense). You could also Suggest
xinit, as it seems like a nice way to start such a minimalistic window
manager." [7 & 8 seem sensible to me. I cannot comment on this one.]
--
To UNSUBSCRIBE, email to debian-wnpp-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/5055c387.9090...@periapt.co.uk
