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

[Antonio Diaz Diaz]

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,
Antonio.