Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 374363 - net-libs/libpri-1.4.12: patch to compile the release version
Summary: net-libs/libpri-1.4.12: patch to compile the release version
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Library (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Tony Vroon (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-07-07 16:12 UTC by Erik Wallin
Modified: 2011-07-08 15:46 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
Modified patch for the released version of 1.4.12 as compared to 1.4.12-beta3 (libpri-1.4.12-released-respect-cflags.patch,1.35 KB, patch)
2011-07-07 21:07 UTC, Erik Wallin
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Erik Wallin 2011-07-07 16:12:54 UTC
The patches for libpri-1.4.12_beta3 need a trivial modification to compile 
libpri-1.4.12. The same ebuild can be used.


Reproducible: Always




Until it makes it into the portage tree, I thought I'd save everyone some time. It compiles, and Asterisk runs, but I haven't done any extensive testing of the stability of the library itself.

Replace file/libpri-1.4.12-respect-cflags.patch by the following modified patch.

new version of libpri-1.4.12-respect-cflags.patch:

--- libpri-1.4.12/Makefile.orig    2011-07-07 16:18:34.185504616 +0200
+++ libpri-1.4.12/Makefile    2011-07-07 16:32:09.095312715 +0200
@@ -90,7 +90,7 @@
     rose_qsig_mwi.lo \
     rose_qsig_name.lo \
     version.lo
-CFLAGS=-Wall -Werror -Wstrict-prototypes -Wmissing-prototypes -g -fPIC $(ALERTING) $(LIBPRI_OPT) $(COVERAGE_CFLAGS)
+CFLAGS=-Wall -Wstrict-prototypes -Wmissing-prototypes -g -fPIC $(ALERTING) $(LIBPRI_OPT) $(COVERAGE_CFLAGS)
 INSTALL_PREFIX=$(DESTDIR)
 INSTALL_BASE=/usr
 LIBDIR=$(INSTALL_PREFIX)$(INSTALL_BASE)/lib
@@ -124,20 +124,12 @@
 
 PRIVERSION:=$(shell GREP=$(GREP) AWK=$(AWK) build_tools/make_version .)
 
-#The problem with sparc is the best stuff is in newer versions of gcc (post 3.0) only.
-#This works for even old (2.96) versions of gcc and provides a small boost either way.
-#A ultrasparc cpu is really v9 but the stock debian stable 3.0 gcc doesnt support it.
-ifeq ($(PROC),sparc64)
-PROC=ultrasparc
-LIBPRI_OPT = -mtune=$(PROC) -O3 -pipe -fomit-frame-pointer -mcpu=v8
+ifneq ($(CODE_COVERAGE),)
+  LIBPRI_OPT=
+  COVERAGE_CFLAGS=-ftest-coverage -fprofile-arcs
+  COVERAGE_LDFLAGS=-ftest-coverage -fprofile-arcs
 else
-  ifneq ($(CODE_COVERAGE),)
-    LIBPRI_OPT=
-    COVERAGE_CFLAGS=-ftest-coverage -fprofile-arcs
-    COVERAGE_LDFLAGS=-ftest-coverage -fprofile-arcs
-  else
-    LIBPRI_OPT=-O2
-  endif
+  LIBPRI_OPT=
 endif
 
 ifeq ($(CPUARCH),i686)
Comment 1 Jaco Kroon 2011-07-07 19:01:01 UTC
Hi,

Can you please upload that as an attachment rather please?
Comment 2 Erik Wallin 2011-07-07 21:07:08 UTC
Created attachment 279363 [details, diff]
Modified patch for the released version of 1.4.12 as compared to 1.4.12-beta3

Better idea to add an attachment. I'm a new to Bugzilla. Don't think there was any way of attaching a file while creating the bug.

Something did eat a +-sign in the cut/paste above, so do use the attachment.

The only real difference to the original patch is that $(LIBPRI_COUNTERS) has been removed from CFLAGS.

The other two patches from 1.4.12-beta3 apply cleanly.
Comment 3 Tony Vroon (RETIRED) gentoo-dev 2011-07-08 15:46:36 UTC
+*libpri-1.4.12 (08 Jul 2011)
+
+  08 Jul 2011; Tony Vroon <chainsaw@gentoo.org> +libpri-1.4.12.ebuild,
+  +files/libpri-1.4.12-released-respect-cflags.patch:
+  Bump to final 1.4.12 release; patches reworked by Erik Wallin from
+  pushtalk.se in bug #374363.