Closed Rhialto closed 1 year ago
I think some of the changes in 2.10pl1
(rather than 2.10
) might make a difference here:
https://gitlab.com/Rhialto/zoo/-/commit/1ca9e2215ddf96a5d3ed41dc02336d293282ed65
in particular looking at __STDC__
in options.h
and in particular setting STDARG
and, I'm guessing, T_SIGNAL
based on that. But those things (everything in options.h
) are of course still better detected in the configure script.
Hi @Rhialto, I've just "recently" adopted zoo and only tested it on Linux so far. There's a LOT of changes and cleanup being made to the code base, and it's far from finished. I'll have a look at your pointers later, if I get some time over (as planned) over the winter holidays.
The 64-bit issue "solved" in the references Debian patch is already fixed in b63c543.
Thanks @troglobit that is great to hear. No rush. One should enjoy one's hobby :-)
@Rhialto there, took me few years to circle back, but it should build better now :-)
Ah yes, indeed it compiles! Great!
I plan to compare your set of patches with my set of patches, to see if there is something in my set that you can still use or need. It will be a bit annoying to do since we begin with slightly different baselines: you from 2.10 and I from 2.10 patchlevel 1. I suspect, for example, that your Debian 01-old-fixes.patch might be derived from this pl1 patch, but who knows :) Also I'm quite sure that both patch sets attack similar sort of issues but likely not in 100% the same way. So this puzzle won't be done tomorrow.
For now, at least, thanks!
I've looked at the packaging of zoo in pkgsrc (see http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/archivers/zoo/?only_with_tag=MAIN ) and it seems that the server which we used for upstream sources (ftp://ftp.kiarchive.ru/pub/unix/arcers/zoo-2.10pl1.tar.gz) has disappeared.
I was considering to switch to your repo even though it contains different patches than pkgsrc does. However, it doesn't compile:
I also noticed patch
3-fix-manage-archive-under-AMD64.patch
(cd71ff845a4951c8921aff1f93c7b3dd89fcaab0) which is a horrible way to fix this 64-bit issue. I did it like this in 2009: https://gitlab.com/Rhialto/zoo/-/commit/7b328789b603c23aa374b0e39ae93f83b717fabd#75e95b0babeec552ef29ad7d4ebf0b62e29a1402_226_226