public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Corinna Vinschen <corinna-cygwin@cygwin.com>
To: cygwin@cygwin.com, newlib@sourceware.org
Subject: Re: GIT source build failure: wcwidth.c::_wcwidth misses __locale_cjk_lang()
Date: Sun, 21 Aug 2016 22:03:00 -0000	[thread overview]
Message-ID: <20160821181524.3vwrbss34nt2wwqi@calimero.vinschen.de> (raw)
In-Reply-To: <ac2b1ee9-4885-c321-e0f9-678acfda2d95@t-online.de>

[-- Attachment #1: Type: text/plain, Size: 1976 bytes --]


[CC newlib mailing list since it affects newlib in the first place]

On Aug 21 14:28, Hans-Bernhard Bröker wrote:
> Am 21.08.2016 um 12:30 schrieb Corinna Vinschen:
> 
> > thanks for the report, but I can't reproduce this.  I can build Cygwin
> > fine natively as well as on Linux, with and without optimization.
> 
> That's strange.  On Re-examination, it builds here, too, even without the
> added include.  I.e. effectively it took only a `touch` of wcwidth.c to fix
> this.
> 
> Could there be a problem with the dependency handling in the newlib auto-foo
> which caused changes to header files to fail triggering a rebuild?

Yes.  If you just pulled and rebuilt, there's a chance that you got
the new headers without triggering a rebuild of dependent sources.

> And while I'm at it: what _is_ the dependency handling here, anyway?  I
> can't seem to find any.  Which could indeed explain the problem.

That's because dependency handling is basically non-existing in newlib.

Nobody added a patch yet to do that.  Newlib being open-source, there's
of course nothing keeping anybody from adding it (*nudge*, *nudge*).

A big problem for many people keeping them from provide patches to the
configury is that newlib's build system still requires the automake
--cygnus option, which isn't supported by automake for quite some time.
The trick is to use an older versions of automake for rebuilding the
configury.  I'm having goods results with this script, which can be
called from newlib or the libc/libm subdirs:

  #!/bin/bash
  aclocal-1.12.2 -I. -I..
  autoconf
  automake-1.12.2 --cygnus --no-force

The autoconf version actually doesn't matter much so I'm using 2.69.

I'm not automake savvy enough to know what to do to get rid of the
--cygnus requirement :(


Corinna

-- 
Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2016-08-21 18:15 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-21  4:01 Hans-Bernhard Bröker
2016-08-21 15:04 ` Corinna Vinschen
2016-08-21 15:25   ` Hans-Bernhard Bröker
2016-08-21 22:03     ` Corinna Vinschen [this message]
2016-08-22  1:31       ` Hans-Bernhard Bröker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160821181524.3vwrbss34nt2wwqi@calimero.vinschen.de \
    --to=corinna-cygwin@cygwin.com \
    --cc=cygwin@cygwin.com \
    --cc=newlib@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).