On Wed, 2006-10-25 at 09:59 +0800, Jedy Wang wrote:
> On Tue, 2006-10-24 at 23:48 +1300, Glynn Foster wrote:
> > Hey Jedy,
> > 
> > Most of this patch didn't actually need a review - it was just a simple 
> > update
> > to the version of the spec, and and some url changes.
> > 
> > Can you rationalize why you're removing the common-lisp directory? What 
> > does it
> > contain, and why we might be advised to remove it?
> I removed it because I did not find /usr/share/common-lisp on Solaris.
libgpg-err installs some files like 
        gpg-error.asd  gpg-error.asd.in  gpg-error-codes.lisp  gpg-error.lisp
gpg-error-package.lisp
under this directory which is used by common-lisp only I think.
> So I think we do not ship it.
> 
> Regards,
> 
> Jedy Wang
> > 
> > 
> > Glynn
> > 
> > Jedy Wang wrote:
> > > Hi all,
> > > 
> > > Please review the patch.
> > > 
> > > Regards,
> > > 
> > > Jedy Wang
> > > 
> > > 
> > > ------------------------------------------------------------------------
> > > 
> > > Index: Solaris/extra-specs/libgpg-error.spec
> > > ===================================================================
> > > --- Solaris/extra-specs/libgpg-error.spec (revision 9565)
> > > +++ Solaris/extra-specs/libgpg-error.spec (working copy)
> > > @@ -3,7 +3,7 @@
> > >  #
> > >  
> > >  Name:         libgpg-error
> > > -Version:      1.3
> > > +Version:      1.4
> > >  Release:      1
> > >  Summary:      libgpg-error - Common error codes for GnuPG, Libgcrypt etc.
> > >  License:      GPL, Other License(s), see package
> > > @@ -11,7 +11,7 @@
> > >  Copyright:    LGPL
> > >  Autoreqprov:  on
> > >  URL:          http://www.gnupg.org/
> > > -Source:       
> > > ftp://ftp.gnupg.org/gcrypt/libgpg-error/%{name}-%{version}.tar.gz
> > > +Source:       
> > > http://ftp.gnupg.org/gcrypt/libgpg-error/%{name}-%{version}.tar.gz
> > >  Source1:      l10n-configure.sh
> > >  BuildRoot:    %{_tmppath}/%{name}-%{version}-build
> > >  
> > > Index: Solaris/extra-specs/libgcrypt.spec
> > > ===================================================================
> > > --- Solaris/extra-specs/libgcrypt.spec    (revision 9565)
> > > +++ Solaris/extra-specs/libgcrypt.spec    (working copy)
> > > @@ -12,14 +12,14 @@
> > >  # usedforbuild    aaa_base acl attr bash bind9-utils bison cpio cpp 
> > > cyrus-sasl db devs diffutils e2fsprogs file filesystem fileutils fillup 
> > > findutils flex gawk gdbm-devel glibc glibc-devel glibc-locale gpm grep 
> > > groff gzip kbd less libgcc libstdc++ libxcrypt m4 make man mktemp 
> > > modutils ncurses ncurses-devel net-tools netcfg pam pam-devel pam-modules 
> > > patch permissions ps rcs readline sed sendmail sh-utils shadow strace 
> > > syslogd sysvinit tar texinfo textutils timezone unitedlinux-release unzip 
> > > util-linux vim zlib-devel autoconf automake binutils bzip2 cracklib gcc 
> > > gdbm gettext libpth libtool perl rpm zlib
> > >  
> > >  Name:         libgcrypt
> > > -Version:      1.2.2
> > > +Version:      1.2.3
> > >  Release:      1
> > >  Summary:      libgcrypt - The GNU crypto library
> > >  License:      GPL, Other License(s), see package
> > >  Group:        Development/Libraries/C and C++
> > >  Autoreqprov:  on
> > >  URL:          http://www.gnupg.org/
> > > -Source:       
> > > ftp://ftp.gnupg.org/gcrypt/libgcrypt/%{name}-%{version}.tar.bz2
> > > +Source:       
> > > http://ftp.gnupg.org/gcrypt/libgcrypt/%{name}-%{version}.tar.bz2
> > >  Source1:      l10n-configure.sh
> > >  BuildRoot:    %{_tmppath}/%{name}-%{version}-build
> > >  
> > > Index: Solaris/SUNWlibgpg-error.spec
> > > ===================================================================
> > > --- Solaris/SUNWlibgpg-error.spec (revision 9565)
> > > +++ Solaris/SUNWlibgpg-error.spec (working copy)
> > > @@ -49,6 +49,8 @@
> > >  #remove this file because SUNWgnome-base-libs deliver same file
> > >  rm $RPM_BUILD_ROOT%{_libdir}/charset.alias
> > >  
> > > +rm -rf $RPM_BUILD_ROOT%{_datadir}/common-lisp
> > > +
> > >  %if %build_l10n
> > >  %else
> > >  # REMOVE l10n FILES
> 


Reply via email to