zsh-workers
 help / color / mirror / code / Atom feed
From: Clint Adams <schizo@debian.org>
To: Andrej Borsenkow <Andrej.Borsenkow@mow.siemens.ru>
Cc: zsh-workers@sunsite.dk
Subject: Re: PATCH: reduce unnecessary linking
Date: Thu, 19 Apr 2001 11:59:08 -0400	[thread overview]
Message-ID: <20010419115908.A27595@dman.com> (raw)
In-Reply-To: <Pine.SV4.4.33.0104191848070.27005-100000@itsrm2.mow.siemens.ru>; from Andrej.Borsenkow@mow.siemens.ru on Thu, Apr 19, 2001 at 06:53:22PM +0400

> What was wrong with AC_SEARCH_LIBS?

It adds the library to LIBS, which would be counterproductive.

> No, that won't do. I (being on SVR4) need '-lsocket -lnsl' for any
> networking code (what our Solaris fellas say?) Moreover, I *do* need them
> both in *main* module and not in any DSO (else some funny effects may
> happen).

I can't compile current CVS on Solaris 8, or I'd check.  -lnsl should
already be in the main module, because that's where yp_all is called.
I'll add -lsocket to zftp.mdd.

Are you saying that all the libraries that any DSO's are linked against
need to be linked to the main binary?

configure:2623: gcc -E -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 conftest.c >/d
ev/null 2>conftest.out
In file included from /usr/include/curses.h:23,
                 from /usr/local/lib/gcc-lib/sparc-sun-solaris2.8/2.95.2/include
/curses.h:5,
                 from configure:2619:
/usr/include/widec.h:38: warning: `getwc' redefined
/usr/include/iso/wchar_iso.h:337: warning: this is the location of the previous 
definition
/usr/include/widec.h:39: warning: `putwc' redefined
/usr/include/iso/wchar_iso.h:340: warning: this is the location of the previous 
definition
/usr/include/widec.h:40: warning: `getwchar' redefined
/usr/include/iso/wchar_iso.h:338: warning: this is the location of the previous 
definition
/usr/include/widec.h:41: warning: `putwchar' redefined
/usr/include/iso/wchar_iso.h:341: warning: this is the location of the previous 
definition
configure: failed program was:
#line 2618 "configure"
#include "confdefs.h"
#include <curses.h>

> I really suggest that this patch should be hold off till 4.0.1 release. It
> is just too dangerous and can break too much.

We should probably start branching in CVS.


  parent reply	other threads:[~2001-04-19 16:00 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-04-19 14:41 Clint Adams
2001-04-19 14:53 ` Andrej Borsenkow
2001-04-19 15:55   ` Andrej Borsenkow
2001-04-19 16:02     ` Clint Adams
2001-04-19 15:59   ` Clint Adams [this message]
2001-04-19 18:15     ` Andrej Borsenkow

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=20010419115908.A27595@dman.com \
    --to=schizo@debian.org \
    --cc=Andrej.Borsenkow@mow.siemens.ru \
    --cc=zsh-workers@sunsite.dk \
    /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.
Code repositories for project(s) associated with this public inbox

	https://git.vuxu.org/mirror/zsh/

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 NNTP newsgroup(s).