Le Thu, 23 Oct 2008 15:09:57 +0200,
Miklos Vajna <[EMAIL PROTECTED]> a écrit :

> On Wed, Oct 22, 2008 at 01:23:49PM +0200, Thomas <[EMAIL PROTECTED]>
> wrote:
> > -build() {
> > -   cd $Fsrcdir || Fdie
> > -   mkdir $Fsrcdir/$pkgname || Fdie
> 
> (...)
> > +           cd $Fsrcdir || Fdie
> > +           mkdir $Fsrcdir/$pkgname
> 
> I don't see the purpose of this change.
> 

I did not change but I just had "#extract tarball:" to explain.

> The Fdie is right there, use "mkdir -p foo || Fdie" if you want to
> allow an incremental build.
> 

> Also, prepending two tabs instead of one makes no sense to me.
> 
Where ? For install demos & documentation ?

> Anyway it looks fine, but please don't push this one as-is now, as the
> patch is quite unreadable because of these whitespace changes.
> 
I thought it could be more readable if I "ventilate" the FrugalBuild :-(

> Please just resend a patch whithout whitespace changes, then probably
> you'll earn my ACK. :)
> 
> PS: I just ignored your second mail, I hope it got it right that it
> was just a duplicate.
Yes
>From fcc5ab0175e14c763e969059a108529fe4f2b605 Mon Sep 17 00:00:00 2001
From: gholafox <[EMAIL PROTECTED]>
Date: Thu, 23 Oct 2008 16:17:31 +0200
Subject: [PATCH] fpc-bin-2.2.2-1-i686
 * Version bump

---
 source/devel-extra/fpc-bin/FrugalBuild |   19 +++++++++++--------
 1 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/source/devel-extra/fpc-bin/FrugalBuild b/source/devel-extra/fpc-bin/FrugalBuild
index 86997a2..4c24dec 100644
--- a/source/devel-extra/fpc-bin/FrugalBuild
+++ b/source/devel-extra/fpc-bin/FrugalBuild
@@ -2,7 +2,7 @@
 # Maintainer: Christian Hamar alias krix <[EMAIL PROTECTED]>
 
 pkgname=fpc-bin
-pkgver=2.0.4
+pkgver=2.2.2
 pkgrel=1
 pkgdesc="FreePascal Compiler (Binary version)"
 url="http://www.freepascal.org";
@@ -13,19 +13,24 @@ groups=('devel-extra')
 up2date="lynx -dump $url |grep 'latest stable' |sed 's/.*Version \(.*\) is.*/\1/'"
 if [ "$CARCH" == "i686" ]; then
 	fpc_arch="i386"
-	sha1sums=('3e3796e4c91709658b007189317ed196fcfce74f')
+	sha1sums=('ea1c5302f61b7d3244f55e773bbf6a81228dc024')
 elif [ "$CARCH" == "x86_64" ]; then
 	fpc_arch="x86_64"
-	sha1sums=('60526f416cbe7b50683def1284d3a0ceec3e7f72')
+	sha1sums=('ea1c5302f61b7d3244f55e773bbf6a81228dc024')
 fi
 source=(ftp://ftp.freepascal.org/pub/fpc/dist/$fpc_arch-linux-$pkgver/`echo $pkgname | sed 's/-bin//'`-$pkgver.$fpc_arch-linux.tar)
 
 build() {
+	#extract tarball:
 	cd $Fsrcdir || Fdie
-	mkdir $Fsrcdir/$pkgname || Fdie
+	mkdir -p $Fsrcdir/$pkgname || Fdie
 	tar -xf binary.$fpc_arch-linux.tar -C $pkgname || Fdie
-	cp doc-pdf.tar.gz $pkgname/doc-pdf.tar.gz || Fdie
-	cp demo.tar.gz $pkgname/demo.tar.gz || Fdie
+	#install documentation:
+	Fmkdir /usr/share/doc/fpc-${pkgver}/demo
+	tar -xf doc-pdf.tar.gz -C ${Fdestdir}/usr/share/doc || Fdie
+	#install demos:
+	tar -xf demo.tar.gz -C ${Fdestdir}/usr/share/doc/fpc-${pkgver}/demo || Fdie
+	#install binaries & other:
 	cd $Fsrcdir/$pkgname || Fdie
 	Fmkdir /usr
 	for i in `ls -1`
@@ -38,5 +43,3 @@ build() {
             Fln /usr/lib/fpc/$pkgver/ppc386 /usr/bin/ppc386
         fi
 }
-
-# optimization OK
-- 
1.6.0.3

_______________________________________________
Frugalware-devel mailing list
[email protected]
http://frugalware.org/mailman/listinfo/frugalware-devel

Reply via email to