Date: Thu, 8 May 2014 10:24:46 +0200 From: =?UTF-8?Q?Bernhard_Fr=C3=B6hlich?= <decke@bluelife.at> To: Melvyn Sopacua <melvyn@magemana.nl> Cc: ports <freebsd-ports@freebsd.org> Subject: Re: Make loosing a variable (emulators/virtualbox-ose-additions) Message-ID: <CAE-m3X1M-RBqhbNx68T-ijWooTVZ5o8aqtyLFqnHrp0cLj%2BfNw@mail.gmail.com> In-Reply-To: <alpine.BSF.2.00.1405042204270.52469@fire.magemana.nl> References: <alpine.BSF.2.00.1405042040050.79927@fire.magemana.nl> <CAE-m3X2q8xLYtEkHZ5dGoxjLGwveFyX-ZVLY=-LuF-sP5MPFzw@mail.gmail.com> <alpine.BSF.2.00.1405042204270.52469@fire.magemana.nl>
next in thread | previous in thread | raw e-mail | index | archive | help
I have just seen that jkim has committed a different fix that should help with that issue. Could you please test to see if it works for you? On Sun, May 4, 2014 at 10:32 PM, Melvyn Sopacua <melvyn@magemana.nl> wrote: > Hi Bernhard, > > On Sun, 4 May 2014, Bernhard Fr=C3=B6hlich wrote: > >> Am 04.05.2014 20:44 schrieb "Melvyn Sopacua" <melvyn@magemana.nl>: >>> >>> >>> Hi, >>> >>> emulators/virtualbox-ose-additions always fails for me in the stage >>> installation, so today I looked a bit further: >>> >>> Bad: >>> install -o root -g wheel -m 444 >>> >> >> /usr/obj/ports/usr/ports/emulators/virtualbox-ose-additions/work/Virtual= Box-4.3.10/out/freebsd.amd64/release/bin/additions/vboxvideo_drv_.so >>> >>> >> >> /usr/obj/ports/usr/ports/emulators/virtualbox-ose-additions/work/stage/u= sr/local/lib/xorg/modules/drivers/vboxvideo_drv.so >>> >>> >>> Good: >>> install -o root -g wheel -m 444 >>> >> >> /usr/obj/ports/usr/ports/emulators/virtualbox-ose-additions/work/Virtual= Box-4.3.10/out/freebsd.amd64/release/bin/additions/vboxvideo_drv_17.so >>> >>> >> >> /usr/obj/ports/usr/ports/emulators/virtualbox-ose-additions/work/stage/u= sr/local/lib/xorg/modules/drivers/vboxvideo_drv.so >>> >>> >>> The difference between bad and good are the 17 missing in the shared >>> object name. The bad line is created in a clean build and the good line >>> if one invokes the install target immediately after that failed one. >>> >>> Is make really loosing a variable here or could this be a parallelizati= on >> >> issue in the upstream build system? >> >> That command is part of the port makefile so it's for sure not an upstre= am >> bug. Sounds like the xserver version that is used there is not read >> properly in some case. > > > I see and makes sense (Mk/bsd.xorg.mk): > .if exists(${LOCALBASE}/bin/X) > XSERVER_VER!=3D ${LOCALBASE}/bin/X -version 2>&1 | sed -n 's;^X\.Org X > Server \([^ ]*\).*;\1;p' > .endif > > ${LOCALBASE}/bin/X won't exist when this is evaluated, unless you cut up > the build stages into a separate make depends followed by make install. > > I believe setting a default that is in sync with x11-servers/xorg-server > in bsd.xorg.mk will fix all cases, since if it doesn't exist the > dependency installed will be provided by that port and if it does, you > got the one that was installed by the user. --=20 Bernhard Fr=C3=B6hlich http://www.bluelife.at/
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAE-m3X1M-RBqhbNx68T-ijWooTVZ5o8aqtyLFqnHrp0cLj%2BfNw>