Date: Thu, 25 Mar 2004 13:21:48 -0500
From: Joe Marcus Clarke <marcus@marcuscom.com>
To: Jeremy Messenger <mezz7@cox.net>
Cc: freebsd-gnome@freebsd.org
Subject: Re: Time to put ${PTHREAD_LIBS} in the x11/xscreensaver-gnome?
Message-ID: <1080238908.781.51.camel@gyros>
In-Reply-To: <opr5fjrfkm8ckrg5@smtp.central.cox.net>
References: <opr5epdtjo8ckrg5@smtp.central.cox.net> <opr5filzl78ckrg5@smtp.central.cox.net> <opr5fjrfkm8ckrg5@smtp.central.cox.net>
next in thread | previous in thread | raw e-mail | index | archive | help
--=-u630zvHmYOeeoYbdL7C9
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable
On Thu, 2004-03-25 at 13:17, Jeremy Messenger wrote:
> On Thu, 25 Mar 2004 13:05:16 -0500, Joe Marcus Clarke=20
> <marcus@marcuscom.com> wrote:
>=20
> > On Thu, 2004-03-25 at 12:52, Jeremy Messenger wrote:
> >> On Thu, 25 Mar 2004 23:46:28 +0900, Alexander Nedotsukov
> >> <bland@FreeBSD.org> wrote:
> >>
> >> > Jeremy Messenger wrote:
> >> >
> >> >> Hello,
> >> >>
> >> >> I do think that GNOME team should add the ${PTHREAD_LIBS} in the
> >> >> x11/xscreensaver-gnome at LDFLAGS line. There are few people have t=
he
> >> >> same problem, so it's not just a Nvidia driver issue. Updated to 4.=
15
> >> >> must have changed something, because the older version doesn't
> >> >> required the ${PTHREAD_LIBS}.
> >> >
> >> > Guys,
> >> >
> >> > I don't think it's good idea because of two reasons:
> >> > 1. I have xscreensaver-demo properly linked and running just fine=20
> >> here.
> >> > Plus since I have nvidia GL library installed when all GL hacks and
> >> > gl-helper mapped through libmap to use libc_r they also runs fine.
> >> > 2. xscreensaver do not make any use of threading at all.
> >> > Even the way you proposing may fix situation on your side this is no=
t
> >> > the right thing. I fact for nvidia users it will be better not to li=
nk
> >> > against any threading library at all. This is the way things also=20
> >> works
> >> > fine on my side.
> >>
> >> If I downgrade to 4.14 or below, then compile it without the
> >> ${PTHREAD_LIBS} and the xscreensaver-gnome works fine.
> >>
> >> > Speaking about incorrect linking. To proove this is the case it's
> >> > necessary to found at least one object wich make use of pthreads API=
=20
> >> but
> >> > do not linked against it or two objects linked against different
> >> > threading libraries. As I said I don't see any.
> >> > Well to be constructive can people (not only once who currently
> >> > expirince a problem with xscreensaver) try the patch attached and=20
> >> report
> >>
> >> See the 'xscreensaver-gnome & threads' thread. We have three people th=
at
> >> who aren't using Nvidia driver.
> >>
> >> We have the same issue, but different video card:
> >>
> >> -Nvidia driver
> >> -ATI radeon 7000 w/ DRI enable
> >> -Radeon 9000 in dual head without DRI
> >> -Unknown, but he doesn't has any Nvidia driver
> >>
> >> > two things is: it works and what is your GL library Nvidia/Mesa?
> >>
> >> Ok, I will test your patch this afternoon.
> >>
> >> > Thanks,
> >> > Alexander.
> >> > ps. Btw, others better follow my advise to figure out what is wrong=20
> >> with
> >> > your installation.
> >>
> >> I doubt there's something wrong with my installtion. Around two to thr=
ee
> >> weeks ago, I formatted my HD and did the clean installtion of=20
> >> 5.2-CURRENT,
> >> then went right straight to update ports tree and grab MarcusCOM CVS f=
or
> >> GNOME 2.5.x. After that, all I did was add debug in the /etc/make.conf=
=20
> >> and
> >> do the 'make install' then install Nvidia driver and the
> >> xscreensaver-gnome doesn't even work by default installtion.
> >>
> >> All above made me believe that there is a new bug in the
> >> xscreensaver-gnome 4.15.
> >
> > The commonality is all affected users are using libc_r on -CURRENT
> > instead of libpthread. There was a change in xscreensaver 4.15 so that
> > libpthread is linked in if found and you're using MesaGL. The problem
> > is, only the GL modules are linked with libpthread.so. This is fine if
> > you're talking about libpthread.so. However, if you're talking about
> > libc_r, then the main binary also has to be linked to libc_r, or you ge=
t
> > the DTP. That said, modifying configure to link ${PTHREAD_LIBS} into
> > all xscreensaver executables should fix this.
>=20
> Weird, I thought libmap.conf can override this when it runs but I guess=20
> not. I saw you have committed a fix, so thanks!
It wasn't a fix, it was just some housekeeping. What happened with
bland's patch?
Joe
>=20
> Cheers,
> Mezz
>=20
> > Joe
> >
> >>
> >> >> Cheers,
> >> >> Mezz
--=20
PGP Key : http://www.marcuscom.com/pgp.asc
--=-u630zvHmYOeeoYbdL7C9
Content-Type: application/pgp-signature; name=signature.asc
Content-Description: This is a digitally signed message part
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (FreeBSD)
iD8DBQBAYyM8b2iPiv4Uz4cRAqG5AJ9KdKkSLgpIKBnL8P3cVnFhScaj7gCfYMKw
0bpty/OrhtRyq0UA6DEDSIQ=
=rdFM
-----END PGP SIGNATURE-----
--=-u630zvHmYOeeoYbdL7C9--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1080238908.781.51.camel>