Hi Carl,

> I recently discovered CTWM as an alternative to TWM.

Welcome to the party   :)


> I did notice a few snags during my first days of usage, one of which
> is that icon names overpaint their containing borders in the icon
> manager. I've searched this mailing list but couldn't find anything
> current pertaining to this issue.
[...]
> For now, I forked the github repo and pushed my change there:

With a little experimentation to figure out where the problem comes
from, and why this fixes it, it looks right to me.  I'll go ahead and
get it in, thanks!


> but before I do anything stupid, I'd like to know if there's
> anything I should pay special attention to.  Formatting and coding
> conventions, review prodecures, what have you.

There are vague scribblings in the direction of a developer's manual
in doc/devman/ in the tree, which includes a bit about code style.
The README tells how to build the HTML output, though it's just
AsciiDoc, so it's easy enough to read the source.  Review procedure is
generally just "bring it up on the list", so you're there.

The git conversion for the mirror isn't really round-trippable, so
pulling changes from that mostly decays to patch application.  For
larger or multi-step changes, it'd be preferable to be able to merge
them with history and attribution etc intact, so it'd be better to
have them in bzr (or brz, nowadays), but for simple stuff there's not
much difference getting a git commit or even just an attached diff.


-- 
Matthew Fuller     (MF4839)   |  [email protected]
Systems/Network Administrator |  http://www.over-yonder.net/~fullermd/
           On the Internet, nobody can hear you scream.

Reply via email to