Re: [PATCH v2] newlib: libm: merge build up a directory

Mike Frysinger <[email protected]>
Newsgroups gmane.comp.lib.newlib
Message-ID <YhP2U+bVoHRdq/cv@vapier>
On 21 Feb 2022 20:24, Corinna Vinschen wrote:
> On Feb 21 19:12, Jon Turney wrote:
> > On 21/02/2022 18:30, Mike Frysinger wrote:
> > > > > > 
> > > > > > Can you please take a look?
> > > > > 
> > > > > how do you build cygwin ?  i've just been doing w/newlib-cygwin git checkout:
> > > > > $ ./configure --target=i686-pc-cygwin && make
> > > > > $ ./configure --target=x86_64-pc-cygwin && make
> > > > > these are passing for me
> > > > 
> > > > You'll need to apply this patch:
> > > > 
> > > > https://cygwin.com/pipermail/cygwin-patches/2022q1/011766.html
> > > > 
> > > > Otherwise, you are linking with a stale libm.a left in your builddir
> > > > from before these changes.
> > > 
> > > while i often test incremental changes, i `rm -rf` the build dir when running
> > > the full test suite to avoid possible issues like this.  and in this case,
> > > both targets pass from a fresh build.
> > 
> > Have you run autogen.sh in the winsup/ directory?  The top-level will just
> > ignore it if there is no ./configure there, and it's not checked in.
> 
> Also, always build in an external build dir.  This may not be a problem
> anymore, but in the olden days a build in the sourcedir was broken.

i always build in sep dir since i'm building ~30 targets in parallel

looks like i needed -U_FORTIFY_SOURCE and --with-cross-bootstrap too.
my patch for libm.a fixes linking of the dll files afaict.  there's
some build errors in the docs that i don't understand, but they don't
seem relevant.
-mike
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEuQK1JxMl+JKsJRrUQWM7n+g39YEFAmIT9lMACgkQQWM7n+g3
9YGTuxAAla3IcAEnQ9pNnLL/q9PEyl7DFau4j4Al+yzYXUzzBoqvyul5XvA4AW8M
4JK5BU+T/lUBJzpxYfWLBrYcHrAQXYf3gCo5G/pAQoAgfRGTfRW6vNkTCUgxkwnx
DyG/51Wcy/DS+CtRBULK7CvDVt770waMjgIKDIc0M1BzE4uU9fA2yCdGjyuQ3idD
+LB5uInyDnC3LDhSJvfi4rwhr7xw2Wr9/w7W7272ToMeIhZcOc4+ee+mxfa6YP3m
1zoN8FYU9sHjxKiw251bd3C8GCSw/QOythWsgQVeRAUnfjbwOgMdsLx+tj9h3rP0
k+ty7j87ucW1HHi2rHv5qzVbil0eg6qjp8gHFqsMLay9z/4NEkVgwVr/aFwKTPah
1CSvwp471pWAT+IgqAYV6AUTyTHdN1WVQT1g0riBWfNsWVBO0zF28v9X2xiji2RX
MSeyTyN4O4gP2T7OA/MSd0aoqvhGYSF8Mz94k63wkuklMF9KPULQ4aHKzh495kif
Ri7j+w9RrQ8KZSR+nSXshEy0zlsFKqQAJs2zSPShuNmg+FyTesspkA/A0ZYkaE8N
aET9XjOkcL9RHW6iLLc4PeNxn1RgqwpyRq+cp044tRnOVwwhmny/chn23Y7X6Ji/
Ocb81HzDloeQ6GsY4rGrSvBENs8PICZPYUz0kn3SLk/2gcPS3NA=
=2tGG
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.