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.

_______________________________________________
Zutils-bug mailing list
Zutils-bug@nongnu.org
http://lists.nongnu.org/mailman/listinfo/zutils-bug

Reply via email to