On Jul 27, 2005, at 09:49:55, Benjamin Reed wrote:
> Kyle Moffett wrote:
>> On a side note, does anyone know if porting "aptitude" (from Debian)
>> has been attempted? I was considering trying such a port and was
>> wondering if anybody has made the attempt before and run into
>> difficult or unsolvable problems.
> I ported it a little while back, it's in unstable. I'd be happy to
> move it to stable if I get a few positive reports. (hint, hint) :)
Actually, after testing it a bit, I've noticed a few issues. Firstly,
It seems to have display glitches occasionally when scrolling through
the list of packages, such that package text is sometimes incorrectly
redrawn, and secondly, it doesn't properly create and open its config
file, I get aptitude config text sent through STDOUT, STDERR, or the
tty whenever I try to change preferences.
After looking through your patch a bit, I see a few potential problems.
When you patch this in configure:
> extern << Your patch omits this line
> #ifdef __cplusplus
> const char *_nl_expand_alias ();
> main ()
> bindtextdomain ("", "");
> return (int) gettext ("") + _nl_msg_cat_cntr + *_nl_expand_alias (0)
> return (int) gettext ("") + _nl_msg_cat_cntr
> return 0;
You delete the line "const char *_nl_expand_alias ()" without deleting
the "extern" and the "#ifdef __cplusplus" above it, which possibly makes
the gettext test fail.
Secondly, the configuration text redirection error comes from this part
of the patch:
> - user_config->Dump(out);
> + user_config->Dump();
I can't see any reason for this change. You don't even change the
argument list for the Dump() function, so I assume it has a default
argument (cout?), but this is definitely not desired behaviour :-D.
-----BEGIN GEEK CODE BLOCK-----
GCM/CS/IT/U d- s++: a18 C++++>$ UB/L/X/*++++(+)>$ P+++(++++)>$ L++++(+
W++(+) N+++(++) o? K? w--- O? M++ V? PS+() PE+(-) Y+ PGP+++ t+(+++) 5
tv-(--) b++++(++) DI+ D+ G e->++++$ h!*()>++$ r !y?(-)
------END GEEK CODE BLOCK------