Hi,

On Tue, Jun 10, 2008 at 07:18:10PM -0400, Jeremy Maitin-Shepard wrote:
> Thanks for creating a Debian package for Conkeror.  Hopefully it will
> help increase the Conkeror userbase.  After looking over the package, I
> have just a few comments:
> 
> 1. The conkeror.xulapp file should not be included, as it is used only
>    for a particular method of installing Conkeror, and is not needed
>    after that.

Already noticed that. This, and some more things of that kind, are
already fixed in my packaging repository[1] and will be fixed with the
next release of the package.

  [1] http://noone.org/hg/conkeror/debian -- I currently use Mercurial
      (hg), but will probably switch to git which can make things
      easier because conkeror itself uses git.

> 2. The -help command-line option is actually handled by xulrunner
>    itself, and merely displays the canned xulrunner usage
>    information.

Seems so, yes.

>    The -width and -height options advertised are not actually
>    functional,

Noticed that, too. It's a known bug in Mozilla since 2000. :-(

>    so they should be removed from the man page. 

I will include hints in the --help output as well as the man page
pointing to the bug reports I posted already to the list.

>    (Many users of Conkeror presumably use a tiling window manager,
>    making the initial size irrelevant, though.)

Ack.

                Regards, Axel
-- 
Axel Beckert - [EMAIL PROTECTED], [EMAIL PROTECTED] - http://noone.org/abe/
_______________________________________________
Conkeror mailing list
[email protected]
https://www.mozdev.org/mailman/listinfo/conkeror

Reply via email to