Mailing List Archive

r1405 - branches/1.0/redhat
Author: ingvar
Date: 2007-05-11 10:20:17 +0200 (Fri, 11 May 2007)
New Revision: 1405

Modified:
branches/1.0/redhat/varnish.spec
branches/1.0/redhat/varnishlog.initrc
Log:
changes from fedora extras review

Modified: branches/1.0/redhat/varnish.spec
===================================================================
--- branches/1.0/redhat/varnish.spec 2007-05-11 08:13:03 UTC (rev 1404)
+++ branches/1.0/redhat/varnish.spec 2007-05-11 08:20:17 UTC (rev 1405)
@@ -1,222 +1,196 @@
-%define lib_name %{name}-libs
-
Summary: Varnish is a high-performance HTTP accelerator
Name: varnish
Version: 1.0.3
-Release: 2
+Release: 7%{?dist}
License: BSD-like
Group: System Environment/Daemons
URL: http://www.varnish-cache.org/
-#Packager: Ingvar Hagelund <ingvar at linpro.no>
-Source0: http://kent.dl.sourceforge.net/sourceforge/varnish/%{name}-%{version}.tar.gz
-BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root
-BuildRequires: gcc gcc-c++ ncurses-devel libtool autoconf automake
-Requires: gcc ncurses kernel >= 2.6.0 %{lib_name} = %version-%{release}
-Vendor: Linpro AS, http://www.linpro.no/
+Source0: http://downloads.sourceforge.net/varnish/varnish-%{version}.tar.gz
+Patch0: varnish-1.0.3.redhat.patch0
+BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
+BuildRequires: ncurses-devel
+Requires: kernel >= 2.6.0 varnish-libs = %{version}-%{release}
+Requires: logrotate
+Requires(post): /sbin/chkconfig
+Requires(preun): /sbin/chkconfig
+Requires(preun): /sbin/service

+# Varnish actually needs gcc installed to work. It uses the C compiler
+# at runtime to compile the VCL configuration files. This is by design.
+Requires: gcc
+
%description
This is the Varnish high-performance HTTP accelerator. Documentation
-and additional information about Varnish is available on the following
-web sites:
+wiki and additional information about Varnish is available on the following
+web site: http://www.varnish-cache.org/

- http://www.varnish-cache.org/ Official web site
- http://varnish.projects.linpro.no/ Developer site and wiki
+%package libs
+Summary: Libraries for %{name}
+Group: System Environment/Libraries
+BuildRequires: ncurses-devel
+#Requires: ncurses

-Technical questions about Varnish and this release should be addressed
-to <varnish-dev at projects.linpro.no>.
+%description libs
+Libraries for %{name}.
+Varnish is a high-performance HTTP accelerator.

-Questions about commercial support and services related to Varnish
-should be addressed to <varnish at linpro.no>.
+## Removed the -devel package for now
+#%package devel
+#Summary: Development libraries for %{name}
+#Group: System Environment/Libraries
+#BuildRequires: ncurses-devel
+#Requires: kernel >= 2.6.0 varnish-libs = %{version}-%{release}
+#
+#%description devel
+#Development libraries for %{name}.
+#Varnish is a high-performance HTTP accelerator

-Copyright (c) 2006 Verdens Gang AS
-Copyright (c) 2006 Linpro AS
-All rights reserved.
-Author: Poul-Henning Kamp <phk at phk.freebsd.dk>
-
-%package -n %{lib_name}
-Summary: Varnish is a high-performance HTTP accelerator
-Group: System Environment/Daemons
-URL: http://www.varnish-cache.org/
-BuildRequires: gcc gcc-c++ ncurses-devel libtool autoconf automake
-Requires: ncurses kernel >= 2.6.0
-Vendor: Linpro AS, http://www.linpro.no/
-
-%description -n %{lib_name}
-The libraries of Varnish, the high-performance HTTP accelerator.
-Documentation and additional information about Varnish is available on
-the following web sites:
-
- http://www.varnish-cache.org/ Official web site
- http://varnish.projects.linpro.no/ Developer site and wiki
-
-Technical questions about Varnish and this release should be addressed
-to <varnish-dev at projects.linpro.no>.
-
-Questions about commercial support and services related to Varnish
-should be addressed to <varnish at linpro.no>.
-
-Copyright (c) 2006 Verdens Gang AS
-Copyright (c) 2006 Linpro AS
-All rights reserved.
-Author: Poul-Henning Kamp <phk at phk.freebsd.dk>
-
-%package -n %{lib_name}-devel
-Summary: Varnish is a high-performance HTTP accelerator
-Group: System Environment/Daemons
-URL: http://www.varnish-cache.org/
-BuildRequires: gcc gcc-c++ ncurses-devel libtool autoconf automake
-Requires: ncurses kernel >= 2.6.0 %{lib_name} = %version-%{release}
-Vendor: Linpro AS, http://www.linpro.no/
-
-%description -n %{lib_name}-devel
-Development files of Varnish, the high-performance HTTP accelerator.
-Documentation and additional information about Varnish is available on
-the following web sites:
-
- http://www.varnish-cache.org/ Official web site
- http://varnish.projects.linpro.no/ Developer site and wiki
-
-Technical questions about Varnish and this release should be addressed
-to <varnish-dev at projects.linpro.no>.
-
-Questions about commercial support and services related to Varnish
-should be addressed to <varnish at linpro.no>.
-
-Copyright (c) 2006 Verdens Gang AS
-Copyright (c) 2006 Linpro AS
-All rights reserved.
-Author: Poul-Henning Kamp <phk at phk.freebsd.dk>
-
%prep
%setup -q
-for i in varnishlog varnishtop varnishd varnishstat varnishhist varnishncsa
+%patch0 -p0
+
+# Convert man pages to UTF-8
+for i in bin/*/*.1 man/*.7
do
- iconv -f iso-8859-1 -t utf-8 < bin/$i/$i.1 > bin/$i/$i.1.utf8
- rm -f bin/$i/$i.1
- mv bin/$i/$i.1.utf8 bin/$i/$i.1
+ iconv -f iso-8859-1 -t utf-8 $i > $i.1.utf8
+ rm -f $i && mv $i.1.utf8 $i
done
-iconv -f iso-8859-1 -t utf-8 < man/vcl.7 > man/vcl.7.utf8
-rm -f man/vcl.7
-mv man/vcl.7.utf8 man/vcl.7

%build
-./autogen.sh
-%configure --sbindir=/usr/sbin
-%{__make}

+# Remove "--disable static" if you want to build static libraries
+# (ie for the devel package)
+%configure --sbindir=/usr/sbin --disable-static
+
+# We have to remove rpath - not allowed in Fedora
+# (This problem only visible on 64 bit arches)
+sed -i 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=""|g' libtool
+sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool
+
+#%{__make} %{?_smp_mflags}
+%{__make}
+
sed -e ' s/8080/80/g ' etc/vcl.conf > redhat/vcl.conf

%install
-rm -rf $RPM_BUILD_ROOT
-%{makeinstall}
-strip %{buildroot}%{_sbindir}/varnishd
-strip %{buildroot}%{_bindir}/varnishhist
-strip %{buildroot}%{_bindir}/varnishlog
-strip %{buildroot}%{_bindir}/varnishncsa
-strip %{buildroot}%{_bindir}/varnishstat
-strip %{buildroot}%{_bindir}/varnishtop
+rm -rf %{buildroot}
+make install DESTDIR=%{buildroot} INSTALL="install -p"

-strip %{buildroot}%{_libdir}/libvarnish.so.0.0.0
-strip %{buildroot}%{_libdir}/libvarnishapi.so.0.0.0
-strip %{buildroot}%{_libdir}/libvcl.so.0.0.0
+# None of these for fedora
+find %{buildroot}/%{_libdir}/ -name '*.la' -exec rm -f {} ';'

-mkdir -p %{buildroot}%{_docdir}/%{name}-%{version}
-mkdir -p %{buildroot}/etc/varnish
-mkdir -p %{buildroot}/etc/init.d
-mkdir -p %{buildroot}/etc/sysconfig
+# Remove this line to build the devel package
+find %{buildroot}/%{_libdir}/ -name '*.so' -type l -exec rm -f {} ';'
+
mkdir -p %{buildroot}/var/lib/varnish
+mkdir -p %{buildroot}/var/log/varnish

-%{__install} -m 0644 INSTALL %{buildroot}%{_docdir}/%{name}-%{version}/INSTALL
-%{__install} -m 0644 LICENSE %{buildroot}%{_docdir}/%{name}-%{version}/LICENSE
-%{__install} -m 0644 README %{buildroot}%{_docdir}/%{name}-%{version}/README
-%{__install} -m 0644 ChangeLog %{buildroot}%{_docdir}/%{name}-%{version}/ChangeLog
-%{__install} -m 0644 redhat/README.redhat %{buildroot}%{_docdir}/%{name}-%{version}/README.redhat
-%{__install} -m 0644 redhat/vcl.conf %{buildroot}%{_docdir}/%{name}-%{version}/vcl.example.conf
-%{__install} -m 0644 redhat/vcl.conf %{buildroot}/etc/varnish/vcl.conf
-%{__install} -m 0644 redhat/varnish.sysconfig %{buildroot}/etc/sysconfig/varnish
-%{__install} -m 0755 redhat/varnish.initrc %{buildroot}/etc/init.d/varnish
+%{__install} -D -m 0644 redhat/vcl.conf %{buildroot}%{_sysconfdir}/varnish/vcl.conf
+%{__install} -D -m 0644 redhat/varnish.sysconfig %{buildroot}%{_sysconfdir}/sysconfig/varnish
+%{__install} -D -m 0644 redhat/varnish.logrotate %{buildroot}%{_sysconfdir}/logrotate.d/varnish
+%{__install} -D -m 0755 redhat/varnish.initrc %{buildroot}%{_sysconfdir}/init.d/varnish
+%{__install} -D -m 0755 redhat/varnishlog.initrc %{buildroot}%{_sysconfdir}/init.d/varnishlog

%clean
-rm -rf $RPM_BUILD_ROOT
+rm -rf %{buildroot}

%files
%defattr(-,root,root,-)
-%{_sbindir}/varnishd
-%{_bindir}/varnishhist
-%{_bindir}/varnishlog
-%{_bindir}/varnishncsa
-%{_bindir}/varnishstat
-%{_bindir}/varnishtop
+%{_sbindir}/*
+%{_bindir}/*
%{_var}/lib/varnish
-%{_mandir}/man1/varnishd.1.gz
-%{_mandir}/man1/varnishhist.1.gz
-%{_mandir}/man1/varnishlog.1.gz
-%{_mandir}/man1/varnishncsa.1.gz
-%{_mandir}/man1/varnishstat.1.gz
-%{_mandir}/man1/varnishtop.1.gz
-%{_mandir}/man7/vcl.7.gz
-%doc %{_docdir}/%{name}-%{version}/INSTALL
-%doc %{_docdir}/%{name}-%{version}/LICENSE
-%doc %{_docdir}/%{name}-%{version}/README
-%doc %{_docdir}/%{name}-%{version}/README.redhat
-%doc %{_docdir}/%{name}-%{version}/ChangeLog
-%doc %{_docdir}/%{name}-%{version}/vcl.example.conf
-%config(noreplace) /etc/varnish/vcl.conf
-%config(noreplace) /etc/sysconfig/varnish
-/etc/init.d/varnish
+%{_var}/log/varnish
+%{_mandir}/man1/*.1*
+%{_mandir}/man7/*.7*
+%doc INSTALL LICENSE README redhat/README.redhat redhat/vcl.conf ChangeLog
+%dir %{_sysconfdir}/varnish/
+%config(noreplace) %{_sysconfdir}/varnish/vcl.conf
+%config(noreplace) %{_sysconfdir}/sysconfig/varnish
+%config(noreplace) %{_sysconfdir}/logrotate.d/varnish
+%{_sysconfdir}/init.d/varnish
+%{_sysconfdir}/init.d/varnishlog

-%files -n %{lib_name}
+%files libs
%defattr(-,root,root,-)
-%{_libdir}/libvarnish.so.0.0.0
-%{_libdir}/libvarnish.so.0
-%{_libdir}/libvarnishapi.so.0.0.0
-%{_libdir}/libvarnishapi.so.0
-%{_libdir}/libvcl.so.0.0.0
-%{_libdir}/libvcl.so.0
-%doc %{_docdir}/%{name}-%{version}/INSTALL
-%doc %{_docdir}/%{name}-%{version}/LICENSE
-%doc %{_docdir}/%{name}-%{version}/README
-%doc %{_docdir}/%{name}-%{version}/README.redhat
-%doc %{_docdir}/%{name}-%{version}/ChangeLog
+%{_libdir}/*.so.*
+%doc LICENSE

-%files -n %{lib_name}-devel
-%defattr(-,root,root,-)
-%{_libdir}/libvarnish.a
-%{_libdir}/libvarnish.la
-%{_libdir}/libvarnish.so
-%{_libdir}/libvarnishapi.a
-%{_libdir}/libvarnishapi.la
-%{_libdir}/libvarnishapi.so
-%{_libdir}/libvcl.a
-%{_libdir}/libvcl.la
-%{_libdir}/libvcl.so
-%doc %{_docdir}/%{name}-%{version}/INSTALL
-%doc %{_docdir}/%{name}-%{version}/LICENSE
-%doc %{_docdir}/%{name}-%{version}/README
-%doc %{_docdir}/%{name}-%{version}/README.redhat
-%doc %{_docdir}/%{name}-%{version}/ChangeLog
+## Removed the -devel package for now
+#%files devel
+#%defattr(-,root,root,-)
+#%{_libdir}/libvarnish.so
+#%{_libdir}/libvarnishapi.so
+#%{_libdir}/libvcl.so

%post
/sbin/chkconfig --add varnish
-/sbin/chkconfig --list varnish

%preun
-/sbin/service varnish stop
-/sbin/chkconfig --del varnish
+if [ $1 -lt 1 ]; then
+ /sbin/service varnish stop > /dev/null 2>/dev/null
+ /sbin/service varnishlog stop > /dev/null 2>/dev/null
+ /sbin/chkconfig --del varnish
+ /sbin/chkconfig --del varnishlog
+fi

-%post -n %{lib_name} -p /sbin/ldconfig
+%postun
+if [ $1 -ge 1 ]; then
+ /sbin/service varnish condrestart > /dev/null 2>/dev/null
+ /sbin/service varnishlog condrestart > /dev/null 2>/dev/null
+fi

-%postun -n %{lib_name} -p /sbin/ldconfig
+%post libs -p /sbin/ldconfig

+%postun libs -p /sbin/ldconfig
+
%changelog
+* Wed May 09 2007 Ingvar Hagelund <ingvar at linpro.no> - 1.0.3-7
+- Simplified the references to the subpackage names
+- Added init and logrotate scripts for varnishlog
+
+* Mon Apr 23 2007 Ingvar Hagelund <ingvar at linpro.no> - 1.0.3-6
+- Removed unnecessary macro lib_name
+- Fixed inconsistently use of brackets in macros
+- Added a condrestart to the initscript
+- All manfiles included, not just the compressed ones
+- Removed explicit requirement for ncurses. rpmbuild figures out the
+ correct deps by itself.
+- Added ulimit value to initskript and sysconfig file
+- Many thanks to Matthias Saou for valuable input
+
+* Mon Apr 16 2007 Ingvar Hagelund <ingvar at linpro.no> - 1.0.3-5
+- Added the dist tag
+- Exchanged RPM_BUILD_ROOT variable for buildroot macro
+- Removed stripping of binaries to create a meaningful debug package
+- Removed BuildRoot and URL from subpackages, they are picked from the
+ main package
+- Removed duplication of documentation files in the subpackages
+- 'chkconfig --list' removed from post script
+- Package now includes _sysconfdir/varnish/
+- Trimmed package information
+- Removed static libs and .so-symlinks. They can be added to a -devel package
+ later if anybody misses them
+
+* Wed Feb 28 2007 Ingvar Hagelund <ingvar at linpro.no> - 1.0.3-4
+- More small specfile fixes for Fedora Extras Package
+ Review Request, see bugzilla ticket 230275
+- Removed rpath (only visible on x86_64 and probably ppc64)
+
+* Tue Feb 27 2007 Ingvar Hagelund <ingvar at linpro.no> - 1.0.3-3
+- Made post-1.0.3 changes into a patch to the upstream tarball
+- First Fedora Extras Package Review Request
+
* Fri Feb 23 2007 Ingvar Hagelund <ingvar at linpro.no> - 1.0.3-2
- A few other small changes to make rpmlint happy
+
* Thu Feb 22 2007 Ingvar Hagelund <ingvar at linpro.no> - 1.0.3-1
- New release 1.0.3. See the general ChangeLog
- Splitted the package into varnish, libvarnish1 and
libvarnish1-devel
+
* Thu Oct 19 2006 Ingvar Hagelund <ingvar at linpro.no> - 1.0.2-7
- Added a Vendor tag
+
* Thu Oct 19 2006 Ingvar Hagelund <ingvar at linpro.no> - 1.0.2-6
- Added redhat subdir to svn
- Removed default vcl config file. Used the new upstream variant instead.
@@ -226,11 +200,14 @@
- Changed the sysconfig script to include a lot more nice features.
Most of these were ripped from the Debian package. Updated initscript
to reflect this.
+
* Tue Oct 10 2006 Ingvar Hagelund <ingvar at linpro.no> - 1.0.1-3
- Moved Red Hat specific files to its own subdirectory
+
* Tue Sep 26 2006 Ingvar Hagelund <ingvar at linpro.no> - 1.0.1-2
- Added gcc requirement.
- Changed to an even simpler example vcl in to /etc/varnish (thanks, perbu)
- Added a sysconfig entry
+
* Fri Sep 22 2006 Ingvar Hagelund <ingvar at linpro.no> - 1.0.1-1
- Initial build.

Modified: branches/1.0/redhat/varnishlog.initrc
===================================================================
--- branches/1.0/redhat/varnishlog.initrc 2007-05-11 08:13:03 UTC (rev 1404)
+++ branches/1.0/redhat/varnishlog.initrc 2007-05-11 08:20:17 UTC (rev 1405)
@@ -32,7 +32,7 @@

if [ "$LOGFILE" = "" ]; then LOGFILE="/var/log/varnish/varnish.log"; fi

-DAEMON_OPTS="-a -w ${LOGFILE} -D -p $PIDFILE"
+DAEMON_OPTS="-a -w ${LOGFILE} -D -P $PIDFILE"

mkdir -p /var/run/varnish 2>/dev/null
r1405 - branches/1.0/redhat [ In reply to ]
ingvar at projects.linpro.no writes:
> Log:
> changes from fedora extras review

For the last time, Ingvar, please to not commit to this branch.

DES
--
Dag-Erling Sm?rgrav
Senior Software Developer
Linpro AS - www.linpro.no