diff mbox

openpgm: bump to version release-5-2-122

Message ID 1445435791-40706-1-git-send-email-Vincent.Riera@imgtec.com
State Superseded
Headers show

Commit Message

Vicente Olivert Riera Oct. 21, 2015, 1:56 p.m. UTC
At the same time perform a GitHub migration, since Google Code will
disappear soon.

- Bump to version release-5-2-122
- Switch to GitHub
- Remove trailing slash from OPENPGM_SUBDIR
- Remove unnecessary python dependencies
- Run the bootstrap.sh script it fails to autoreconf properly
- Add a hash file
- Rewrap the help text to 72 characters length
- Change the project URL

Signed-off-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
---
 package/openpgm/Config.in    |   13 +++++++------
 package/openpgm/openpgm.hash |    2 ++
 package/openpgm/openpgm.mk   |   17 +++++++++++------
 3 files changed, 20 insertions(+), 12 deletions(-)
 create mode 100644 package/openpgm/openpgm.hash

Comments

Vicente Olivert Riera Oct. 21, 2015, 2 p.m. UTC | #1
On 10/21/2015 02:56 PM, Vicente Olivert Riera wrote:
[snip]
> - Run the bootstrap.sh script it fails to autoreconf properly
- Run the bootstrap.sh script; it fails to autoreconf properly

(missing semicolon to separate)
[snip]
Thomas Petazzoni Oct. 21, 2015, 3:15 p.m. UTC | #2
Dear Vicente Olivert Riera,

On Wed, 21 Oct 2015 15:00:52 +0100, Vicente Olivert Riera wrote:
> On 10/21/2015 02:56 PM, Vicente Olivert Riera wrote:
> [snip]
> > - Run the bootstrap.sh script it fails to autoreconf properly
> - Run the bootstrap.sh script; it fails to autoreconf properly

I quickly looked at the bootstrap.sh script, and besides the messages in
Japanese (or what I guessed to be Japanese), nothing really unusual
stands out. What fails if you just have AUTORECONF = YES ?

Thomas
Vicente Olivert Riera Oct. 21, 2015, 3:42 p.m. UTC | #3
Dear Thomas Petazzoni,

On 10/21/2015 04:15 PM, Thomas Petazzoni wrote:
> Dear Vicente Olivert Riera,
> 
> On Wed, 21 Oct 2015 15:00:52 +0100, Vicente Olivert Riera wrote:
>> On 10/21/2015 02:56 PM, Vicente Olivert Riera wrote:
>> [snip]
>>> - Run the bootstrap.sh script it fails to autoreconf properly
>> - Run the bootstrap.sh script; it fails to autoreconf properly
> 
> I quickly looked at the bootstrap.sh script, and besides the messages in
> Japanese (or what I guessed to be Japanese), nothing really unusual
> stands out. What fails if you just have AUTORECONF = YES ?

This is the failure:

>>> openpgm release-5-2-122 Configuring
>>> openpgm release-5-2-122 Autoreconfiguring
aclocal: error: couldn't open directory 'm4': No such file or directory
autoreconf: /br/output/host/usr/bin/aclocal failed with exit status: 1

With AUTORECONF = YES, we start running aclocal. I think it normally
starts running libtoolize, but since the m4 directory doesn't exist, it
starts directly with aclocal.

With the bootstrap script work because it will call libtoolize despite
of the m4 directory not existing.

If you prefer, this approach also works, although libtoolize is ran
twice (first by the hook, and then by autoreconf):

(I keep the line numbers to preserve the indentation)

13 OPENPGM_AUTORECONF = YES
14
15 # we need to run libtoolize before autoreconf, so the missing m4
16 # directory is created.
17 define OPENPGM_LIBTOOLIZE
18         cd $(@D)/$(OPENPGM_SUBDIR) && PATH=$(BR_PATH) libtoolize
19 endef
20 OPENPGM_PRE_CONFIGURE_HOOKS += OPENPGM_LIBTOOLIZE

Regards,

Vincent.

> 
> Thomas
>
Thomas Petazzoni Oct. 21, 2015, 3:53 p.m. UTC | #4
Dear Vicente Olivert Riera,

On Wed, 21 Oct 2015 16:42:59 +0100, Vicente Olivert Riera wrote:

> 13 OPENPGM_AUTORECONF = YES
> 14
> 15 # we need to run libtoolize before autoreconf, so the missing m4
> 16 # directory is created.
> 17 define OPENPGM_LIBTOOLIZE
> 18         cd $(@D)/$(OPENPGM_SUBDIR) && PATH=$(BR_PATH) libtoolize
> 19 endef
> 20 OPENPGM_PRE_CONFIGURE_HOOKS += OPENPGM_LIBTOOLIZE

Why not just:

define OPENPGM_CREATE_M4_DIR
	mkdir -p $(@D)/$(OPENPGM_SUBDIR)/m4
endef
OPENPGM_POST_PATCH_HOOKS += OPENPGM_CREATE_M4_DIR

which works (tested locally).

Thanks,

Thomas
diff mbox

Patch

diff --git a/package/openpgm/Config.in b/package/openpgm/Config.in
index 3e90422..d541d13 100644
--- a/package/openpgm/Config.in
+++ b/package/openpgm/Config.in
@@ -6,13 +6,14 @@  config BR2_PACKAGE_OPENPGM
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	depends on BR2_USE_WCHAR
 	help
-	  OpenPGM is an open source implementation of the Pragmatic General
-	  Multicast (PGM) specification in RFC 3208 available at www.ietf.org.
-	  PGM is a reliable and scalable multicast protocol that enables
-	  receivers to detect loss, request retransmission of lost data, or
-	  notify an application of unrecoverable loss.
+	  OpenPGM is an open source implementation of the Pragmatic
+	  General Multicast (PGM) specification in RFC 3208 available at
+	  www.ietf.org. PGM is a reliable and scalable multicast
+	  protocol that enables receivers to detect loss, request
+	  retransmission of lost data, or notify an application of
+	  unrecoverable loss.
 
-	  http://code.google.com/p/openpgm/
+	  https://github.com/steve-o/openpgm/
 
 comment "openpgm needs a toolchain w/ wchar, threads"
 	depends on !(BR2_TOOLCHAIN_HAS_THREADS && BR2_USE_WCHAR)
diff --git a/package/openpgm/openpgm.hash b/package/openpgm/openpgm.hash
new file mode 100644
index 0000000..f20e02a
--- /dev/null
+++ b/package/openpgm/openpgm.hash
@@ -0,0 +1,2 @@ 
+# Locally calculated
+sha256 ecd48fd0bcfd8351ce6dcf31b063dd8fe83d3dab41bca62a6ebd6880a88c4a69  openpgm-release-5-2-122.tar.gz
diff --git a/package/openpgm/openpgm.mk b/package/openpgm/openpgm.mk
index d54e8eb..bb3105a 100644
--- a/package/openpgm/openpgm.mk
+++ b/package/openpgm/openpgm.mk
@@ -4,15 +4,20 @@ 
 #
 ################################################################################
 
-OPENPGM_VERSION = 5.1.118~dfsg
-OPENPGM_SOURCE = libpgm-$(OPENPGM_VERSION).tar.gz
-OPENPGM_SITE = http://openpgm.googlecode.com/files
+OPENPGM_VERSION = release-5-2-122
+OPENPGM_SITE = $(call github,steve-o,openpgm,$(OPENPGM_VERSION))
 OPENPGM_LICENSE = LGPLv2.1+
 OPENPGM_LICENSE_FILES = openpgm/pgm/LICENSE
 OPENPGM_INSTALL_STAGING = YES
-OPENPGM_AUTORECONF = YES
-OPENPGM_SUBDIR = openpgm/pgm/
-OPENPGM_DEPENDENCIES = $(if $(BR2_PACKAGE_PYTHON3),host-python3,host-python)
+OPENPGM_SUBDIR = openpgm/pgm
+
+# openpgm doesn't autoreconf properly, we have to bootstrap it.
+OPENPGM_DEPENDENCIES += host-automake host-autoconf host-libtool
+define OPENPGM_BOOTSTRAP
+	cd $(@D)/$(OPENPGM_SUBDIR) && PATH=$(BR_PATH) ./bootstrap.sh
+endef
+OPENPGM_PRE_CONFIGURE_HOOKS += OPENPGM_BOOTSTRAP
+
 OPENPGM_CONF_ENV = \
 	ac_cv_file__proc_cpuinfo=yes \
 	ac_cv_file__dev_rtc=no \