This is the mail archive of the
cygwin-apps@cygwin.com
mailing list for the Cygwin project.
Re: Ready for test coreutils-5.2.0-1
- From: Joshua Daniel Franklin <joshuadfranklin at yahoo dot com>
- To: cygwin-apps at cygwin dot com
- Date: Thu, 11 Mar 2004 22:45:51 -0600
- Subject: Re: Ready for test coreutils-5.2.0-1
- References: <4050DE0B.3030904@fangorn.ca>
Mark,
Thanks for packaging this and being willing to maintain it.
Comments below.
On Thu, Mar 11, 2004 at 04:45:47PM -0500, Mark Blackburn wrote:
> I've created a new coreutils package that I hope addresses some of the
> concerns recently voiced about it:
>
> -cp.exe corrupting files when copying from binary to text mounts.
> -printf.exe (and others) being statically linked with libintl.a and
> libiconv.a created large binaries.
>
> http://blackburn.homeip.net/cygwin-packages/release/coreutils/coreutils-5.2.0-1-src.tar.bz2
> http://blackburn.homeip.net/cygwin-packages/release/coreutils/coreutils-5.2.0-1.tar.bz2
> http://blackburn.homeip.net/cygwin-packages/release/coreutils/setup.hint
>
> 5a9ee1e8ef9455a14b9ed52c0f52e03e *coreutils-5.2.0-1-src.tar.bz2
> 779d8871f359223c2337a7ead83e2b9f *coreutils-5.2.0-1.tar.bz2
>
>
> There are also some patches I'm considering adding from fileutils-4.1-2
> (proposed-fileutils-patches.txt). I got these by diffing fileutils-4.1
> with the src package for fileutils-4.1-2. I was hoping that the previous
> maintainer could comment on these patches so I could figure out if they
> are relevant for the coreutils package.
If they the changes haven't been made in the coreutils source, why
not continue using the Cygwin-specific patches?
There are still conflicting binaries:
kill.exe : cygwin-1.5.7-1
readlink.exe : cygutils-1.2.4-1
uptime.exe : procps-010801-2
I think the way other packages have dealt with this problem is by
renaming with a prefix ("corekill" like "prockill") or just
not installing them. I don't want to speak for Chuck, but the
other option is, if he gets a new cygutils ready with Harold's
mkshortcut patch he could also remove readlink from cygutils.
As for obsoleting {file,sh-,text}utils, what we did with
newlib-man->cygwin-doc was create an empty newlib-man-$NEWVERISON
and put that in cygwin-doc's requires. That could work here.
Speaking of setups.hint, coreutils' ldesc starts with "A command line
file transfer program. It suppore Utilities" ???
Other than that, the packages looks good. My basic see-if-it-runs
tests worked (for i in *exe; do ./$i --help >> foo; done), and the
source package builds the binary package.
By the way, downloading the packages was really slow for me. I took
the liberty of putting them here for any other potential reviewers:
<http://ns1.iocc.com/~joshua/cygwin/coreutils-5.2.0-1-src.tar.bz2>
<http://ns1.iocc.com/~joshua/cygwin/coreutils-5.2.0-1.tar.bz2>
<http://ns1.iocc.com/~joshua/cygwin/setup.hint>
----snip----
#!/bin/sh
wget http://ns1.iocc.com/~joshua/cygwin/coreutils-5.2.0-1-src.tar.bz2
wget http://ns1.iocc.com/~joshua/cygwin/coreutils-5.2.0-1.tar.bz2
wget http://ns1.iocc.com/~joshua/cygwin/setup.hint
----snip----