[Ltib] Re: Building xorg-x11-proto-devel: Minor proposed fix to latest change in spec file.

[Stuart Hughes]

Hi Rodney,

Thanks for double checking.  I think it's best to leave it for now.
However looking at it the pushd/popd should be replaced at some point to
be compatible with dash (Ubuntu), I recall having to do something
similar for another package the other week.  I mention this in case
someone else runs into this.

Regards, Stuart

Rodney Lott wrote:
> Hi, Stuart.
> 
> I just did the same test as you and got the same results!  :-S
> 
> I guess my system was either in a bad state or I gave results in
> mid-machinations.  Sorry about that.  I'll try to do a clean build a
> couple times before posting in future. 
> 
> At any rate, given that the directory only contains subdirectories,
> either one will do.  I'll just revert my change then.  No point
> disturbing something if it isn't really broken.
> 
> Thanks.
> 
> Rodney
> 
> -----Original Message-----
> From: Stuart Hughes [mailto:address@hidden
> Sent: Thu 11/18/2010 5:51 PM
> To: Rodney Lott
> Cc: address@hidden
> Subject: Re: Building xorg-x11-proto-devel: Minor proposed fix to latest
> change in spec file.
> 
> Hi Rodney,
> 
> Thanks for the patch, which looks okay, however before I apply it, I'm
> trying to understand why at the point that code runs there's anything
> except directories.
> 
> I just ran:
> 
> ./ltib -p xorg-x11-proto-devel -m prep
> ./ltib -p xorg-x11-proto-devel -m scbuild
> cd rpm/BUILD/xorg-x11-proto-devel-7.3/
> 
> and at that point if I do: ls -1 I see:
> 
> bigreqsproto-1.0.2/
> compositeproto-0.4/
> damageproto-1.1.0/
> dmxproto-2.2.2/
> dri2proto-1.1/
> evieext-1.0.2/
> fixesproto-4.0/
> fontcacheproto-0.1.2/
> fontsproto-2.0.2/
> glproto-1.4.9/
> inputproto-1.4.3/
> kbproto-1.0.3/
> randrproto-1.2.1/
> recordproto-1.13.2/
> renderproto-0.9.3/
> resourceproto-1.0.2/
> scrnsaverproto-1.1.0/
> trapproto-3.4.3/
> videoproto-2.2.2/
> xcmiscproto-1.1.2/
> xextproto-7.0.2/
> xf86bigfontproto-1.1.2/
> xf86dgaproto-2.0.3/
> xf86driproto-2.0.4/
> xf86miscproto-0.9.2/
> xf86rushproto-1.1.2/
> xf86vidmodeproto-2.2.2/
> xineramaproto-1.1.2/
> xproto-7.0.12/
> xproxymanagementprotocol-1.0.2/
> 
> 
> So I'm wondering why on your build you're seeing the weird:
>         pushd COPYING-bigreqsproto
> 
> Regards, Stuart
> 
> Rodney Lott wrote:
>> Hi, Stuart.
>>
>> Here's a fix to the error message I found after fetching the fix to the
>> xorg-x11-proto-devel.spec file:
>>
>> Index: dist/lfs-5.1/xorg-x11-proto-devel/xorg-x11-proto-devel.spec
>> ===================================================================
>> RCS file:
>>
> /sources/ltib/ltib/dist/lfs-5.1/xorg-x11-proto-devel/xorg-x11-proto-devel.spec,v
>> retrieving revision 1.1.1.1
>> diff -a -u -r1.1.1.1 xorg-x11-proto-devel.spec
>> --- dist/lfs-5.1/xorg-x11-proto-devel/xorg-x11-proto-devel.spec 5 Dec
>> 2008 14:25:45 -0000   1.1.1.1
>> +++ dist/lfs-5.1/xorg-x11-proto-devel/xorg-x11-proto-devel.spec 15 Nov
>> 2010 16:55:34 -0000
>> @@ -57,7 +57,7 @@
>>
>>  %Build
>>  # Proceed through each proto package directory, building them all
>> -for dir in $(ls -1) ; do
>> +for dir in $(ls -p | grep "/") ; do
>>     pushd $dir
>>     ./configure --prefix=%{_prefix} --host=$CFGHOST --build=%{_build}
>>     make
>> @@ -68,7 +68,7 @@
>>
>>  %Install
>>  rm -rf $RPM_BUILD_ROOT
>> -for dir in $(ls -1) ; do
>> +for dir in $(ls -p | grep "/") ; do
>>     pushd $dir
>>     make install DESTDIR=$RPM_BUILD_ROOT/%{pfx}
>>     install -m 444 COPYING-${dir%%-*} $OLDPWD
>>
>> The "ls -1" will include files and directories, so using "ls -p | grep
>> "/"" will just grab the directories only.  I am sure there is probably a
>> better way of doing this, but this seems to work for me.
>>
>> Thanks.
>>
>>
>>
>>
>> -----Original Message-----
>> From: Stuart Hughes [mailto:address@hidden
>> Sent: Sat 11/13/2010 2:16 PM
>> To: Rodney Lott
>> Subject: Re: Builing full xorg-server: Is there a xf86dgaproto package
>>
>> Hi Rodney,
>>
>> Unfortunately the full xorg-server build is not as well tested as Kdrive.
>>
>> xorg-x11-proto-devel.spec provides xf86dgaproto, but the build order of
>> these package was wrong (it needs to be built before xorg-server).
>>
>> I have checked-in a fix for this, if you do a 'cvs up -dP' and re-run
>> your build it should work (I just tried).
>>
>> Regards, Stuart
>>
>>
>> Rodney Lott wrote:
>>> Hi, Stuart.
>>>
>>> As you suggested, I updated my tarball to the latest in CVS and started
>>> to rebuild things, which for me includes the xorg-server package.    I
>>> got to this stage in the xorg-server configure step, when it failed at
>>> this point:
>>>
>>> checking for DGA... configure: error: Package requirements
>>> (xf86dgaproto) were not met:
>>>
>>> No package 'xf86dgaproto' found
>>>
>>> Consider adjusting the PKG_CONFIG_PATH environment variable if you
>>> installed software in a non-standard prefix.
>>>
>>> Alternatively, you may set the environment variables DGA_CFLAGS
>>> and DGA_LIBS to avoid the need to call pkg-config.
>>> See the pkg-config man page for more details.
>>>
>>> Is there a dgaproto package that would fill this requirement?  I see
>>> there is a package called libXxf86dga, but I am not sure if this is the
>>> correct one.  If there is, let me know.  If not, then I will proceed in
>>> making one.
>>>
>>> Thanks.
>>>
>>> Rodney
>>>
>>
>