Hello community, here is the log from the commit of package alpine for openSUSE:Factory checked in at 2019-09-05 12:42:53 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/alpine (Old) and /work/SRC/openSUSE:Factory/.alpine.new.7948 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "alpine" Thu Sep 5 12:42:53 2019 rev:45 rq:728163 version:unknown Changes: -------- --- /work/SRC/openSUSE:Factory/alpine/alpine.changes 2019-05-02 19:22:21.633912255 +0200 +++ /work/SRC/openSUSE:Factory/.alpine.new.7948/alpine.changes 2019-09-05 12:42:55.423472189 +0200 @@ -1,0 +2,6 @@ +Mon Aug 26 08:33:31 UTC 2019 - Jan Engelhardt <jeng...@inai.de> + +- Add return-values.diff to unbreak build. +- Use more macros for standard dirs in the build recipe. + +------------------------------------------------------------------- @@ -325 +330,0 @@ - New: ---- return-values.diff ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ alpine.spec ++++++ --- /var/tmp/diff_new_pack.21EAat/_old 2019-09-05 12:42:57.347471812 +0200 +++ /var/tmp/diff_new_pack.21EAat/_new 2019-09-05 12:42:57.347471812 +0200 @@ -24,7 +24,7 @@ Group: Productivity/Networking/Email/Clients Version: 2.21 Release: 0 -Url: http://alpine.x10host.com/alpine/ +URL: http://alpine.x10host.com/alpine/ # direct download does not work for openSUSE:Factory # Source: http://patches.freeiz.com/alpine/release/src/%name-%version.tar.xz @@ -40,6 +40,7 @@ Patch10: pico-fix-spurious-undef-warnings.diff Patch20: pine-expression-warnings.diff Patch60: signal-and-panic-improvements.diff +Patch61: return-values.diff # # Eduardo Chappa's patches. # http://patches.freeiz.com/alpine/ @@ -52,7 +53,6 @@ Patch614: chappa-fillpara.patch Patch615: chappa-fromheader.patch Patch616: chappa-rules.patch -BuildRoot: %{_tmppath}/%{name}-%{version}-build BuildRequires: autoconf >= 2.69 BuildRequires: krb5-devel BuildRequires: libtool @@ -136,6 +136,7 @@ %patch10 -p1 %patch20 -p1 %patch60 -p1 +%patch61 -p1 %endif # End of "if !{build_vanilla}" %build @@ -166,10 +167,10 @@ : autoreconf -fiv %configure \ - --with-ssl-dir=/usr \ - --with-smtp-msa=/usr/sbin/sendmail \ - --with-password-prog=/usr/bin/passwd \ - --with-npa=/usr/bin/inews \ + --with-ssl-dir="%{_prefix}" \ + --with-smtp-msa="%{_sbindir}/sendmail" \ + --with-password-prog="%{_bindir}/passwd" \ + --with-npa="%{_bindir}/inews" \ --with-spellcheck-prog="please set Speller to eg. 'aspell -c' in SETUP/Configuration"\ --with-system-pinerc=%_sysconfdir/pine.conf \ --with-system-fixed-pinerc=%_sysconfdir/pine.conf.fixed \ @@ -199,17 +200,17 @@ # # When called as alpinef, alpine uses function keys instead of Control keys: # -ln $RPM_BUILD_ROOT%{_bindir}/alpine $RPM_BUILD_ROOT%{_bindir}/alpinef -install -m755 imap/mailutil/mailutil $RPM_BUILD_ROOT%{_bindir} -install -m644 imap/src/mailutil/mailutil.1 $RPM_BUILD_ROOT%{_mandir}/man1/ -install -D -m644 %{SOURCE1} $RPM_BUILD_ROOT/usr/share/pixmaps/%name.png -install -D -m644 %{SOURCE2} $RPM_BUILD_ROOT/usr/share/applications/%name.desktop +ln %{buildroot}/%{_bindir}/alpine %{buildroot}/%{_bindir}/alpinef +install -m755 imap/mailutil/mailutil %{buildroot}/%{_bindir} +install -m644 imap/src/mailutil/mailutil.1 %{buildroot}/%{_mandir}/man1/ +install -D -m644 %{SOURCE1} %{buildroot}/%{_datadir}/pixmaps/%name.png +install -D -m644 %{SOURCE2} %{buildroot}/%{_datadir}/applications/%name.desktop %suse_update_desktop_file %name : -ln -sf alpine $RPM_BUILD_ROOT%{_bindir}/pine +ln -sf alpine %{buildroot}/%{_bindir}/pine : -install -m755 pico/{pico,pilot} $RPM_BUILD_ROOT%{_bindir} -install -m644 doc/man1/{pico.1,pilot.1} $RPM_BUILD_ROOT%{_mandir}/man1/ +install -m755 pico/{pico,pilot} %{buildroot}/%{_bindir} +install -m644 doc/man1/{pico.1,pilot.1} %{buildroot}/%{_mandir}/man1/ %check #since where are no logs in the package at the moment, there are no checks, @@ -265,7 +266,6 @@ fi %files -%defattr(-, root, root) %doc %{_mandir}/man1/alpine.* %doc %{_mandir}/man1/rpdump.* %doc %{_mandir}/man1/rpload.* @@ -276,16 +276,14 @@ %{_bindir}/alpinef %{_bindir}/rp* %{_bindir}/mailutil -/usr/share/applications/%name.desktop -/usr/share/pixmaps/%name.png +%{_datadir}/applications/%name.desktop +%{_datadir}/pixmaps/%name.png %files -n pico -%defattr(-, root, root) %{_bindir}/pico %doc %{_mandir}/man1/pico.* %files -n pilot -%defattr(-, root, root) %{_bindir}/pilot %doc %{_mandir}/man1/pilot.* ++++++ return-values.diff ++++++ From: Jan Engelhardt <jeng...@inai.de> Date: 2019-08-26 10:33:27.198383143 +0200 openSUSE uses -Werror=return-type since a while... :-/ Returning a value in a void function has no practical issues, unlike returning no value in an int function. But gcc does not differentiate between the two cases. [ 139s] send.c:4200:12: error: 'return' with a value, in function returning void [-Werror=return-type] [ 139s] 4200 | return 0; --- pith/send.c | 1 - 1 file changed, 1 deletion(-) Index: alpine-2.21/pith/send.c =================================================================== --- alpine-2.21.orig/pith/send.c +++ alpine-2.21/pith/send.c @@ -4197,7 +4197,6 @@ set_parameter(PARAMETER **param, char *p pm->value = cpystr(new_value); } } - return 0; }