[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Zutils-bug] Some issues on Solaris (patches referenced)

From: Antonio Diaz Diaz
Subject: Re: [Zutils-bug] Some issues on Solaris (patches referenced)
Date: Fri, 19 Nov 2010 21:15:15 +0100
User-agent: Mozilla/5.0 (X11; U; Linux i586; en-US; rv:1.7.11) Gecko/20050905

Hello Dagobert,

Dagobert Michelsen wrote:
I compiled zutils 0.8 for Solaris and some patches are needed which you may
want to adopt / adjust for proper detection.

Thanks for reporting this. I'll comment on each patch separately.

On Solaris there is no stdint.h, but inttypes.h.

But I neither need nor want all the crap of inttypes.h included in zutils. I think it is time for Solaris to start implementing stdint.h.

Additionally, signal.h is needed in some files.

Those files already #include <csignal>. Why is signal.h also needed?

On Solaris only /usr/ucb/echo understands -n, /usr/bin/echo does not
(the /opt/csw part can be ignored for a general patch).

I have already removed "echo -n" from all my projects. Surely this patch is no more needed.

The renaming of the utilities to an uppercase "Z" because of the gzip
collision is disturbing and I would appreciate a solution between you and
the gzip maintainer.

Rewriting zutils in C++ is the first step towards such a solution. As soon as I am confident that the new zutils work reasonably well, I'll write again to the gzip list. Stay tuned. :-)

The call can be applied to two function signatures and should be made unique:
"zcat.cc", line 48: Error: Overloading ambiguity between "std::string::insert(char*, unsigned, 
char)" and "std::string::insert(unsigned, unsigned, char)".

Please, can you see if using the suffix 'U' like in the following line works?
    else str.insert( 0U, 1, '1' );

Best regards,

reply via email to

[Prev in Thread] Current Thread [Next in Thread]