[asterisk-users] offline builds - mp3 [patch]

2013-06-04 Thread Daniel Pocock


As mentioned in the thread about MP3, I found that the rpmbuild process
demands network access, e.g. to access the mp3 code in SVN.

Some people need to build on isolated networks though

I've attached a patch that allows the MP3 code to be placed in /tmp
before the build starts, then svn will not be used during the build.  If
it finds /tmp/asterisk-contrib-mp3.tar.gz then it will be used instead
of going to SVN

I'm not sure if there are other build steps that access the network,
this one was more obvious because I was trying to build on a fresh VM
without any svn client




--- contrib/scripts/get_mp3_source.sh.orig	2013-06-04 12:41:08.222602824 +0200
+++ contrib/scripts/get_mp3_source.sh	2013-06-04 12:40:45.218602846 +0200
@@ -9,6 +9,15 @@
 exit 1
 fi
 
+LOCAL_COPY=/tmp/asterisk-contrib-mp3.tar.gz
+if [ -f ${LOCAL_COPY} ]; then
+echo ***
+echo Found ${LOCAL_COPY} - unpacking it, not downloading
+echo ***
+tar xzf ${LOCAL_COPY}
+exit 0
+fi
+
 svn export http://svn.digium.com/svn/thirdparty/mp3/trunk addons/mp3 $@
 
 exit 0
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
New to Asterisk? Join us for a live introductory webinar every Thurs:
   http://www.asterisk.org/hello

asterisk-users mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-users

Re: [asterisk-users] offline builds - mp3 [patch]

2013-06-04 Thread Tzafrir Cohen
On Tue, Jun 04, 2013 at 12:49:35PM +0200, Daniel Pocock wrote:
 
 
 As mentioned in the thread about MP3, I found that the rpmbuild process
 demands network access, e.g. to access the mp3 code in SVN.
 
 Some people need to build on isolated networks though
 
 I've attached a patch that allows the MP3 code to be placed in /tmp
 before the build starts, then svn will not be used during the build.  If
 it finds /tmp/asterisk-contrib-mp3.tar.gz then it will be used instead
 of going to SVN
 
 I'm not sure if there are other build steps that access the network,
 this one was more obvious because I was trying to build on a fresh VM
 without any svn client

I'm sure you're aware of:
http://patch-tracker.debian.org/patch/series/view/asterisk/1:1.8.13.1~dfsg-3/mpglib

-- 
   Tzafrir Cohen
icq#16849755  jabber:tzafrir.co...@xorcom.com
+972-50-7952406   mailto:tzafrir.co...@xorcom.com
http://www.xorcom.com  iax:gu...@local.xorcom.com/tzafrir

--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
New to Asterisk? Join us for a live introductory webinar every Thurs:
   http://www.asterisk.org/hello

asterisk-users mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-users


Re: [asterisk-users] offline builds - mp3 [patch]

2013-06-04 Thread Daniel Pocock
On 04/06/13 18:37, Tzafrir Cohen wrote:
 On Tue, Jun 04, 2013 at 12:49:35PM +0200, Daniel Pocock wrote:


 As mentioned in the thread about MP3, I found that the rpmbuild process
 demands network access, e.g. to access the mp3 code in SVN.

 Some people need to build on isolated networks though

 I've attached a patch that allows the MP3 code to be placed in /tmp
 before the build starts, then svn will not be used during the build.  If
 it finds /tmp/asterisk-contrib-mp3.tar.gz then it will be used instead
 of going to SVN

 I'm not sure if there are other build steps that access the network,
 this one was more obvious because I was trying to build on a fresh VM
 without any svn client
 
 I'm sure you're aware of:
 http://patch-tracker.debian.org/patch/series/view/asterisk/1:1.8.13.1~dfsg-3/mpglib
 

The notes suggest that MP3 patent issues are a factor so I guessed
that's why it is excluded from the tarball

When building with rpmbuild the tarball is usually not unpacked
manually, hence my own proposed patch looks in /tmp for the mp3 code -
it could just as easily use your the patch from Debian as an input
though, as long as it can be found in /tmp or some other predefined
location.



--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
New to Asterisk? Join us for a live introductory webinar every Thurs:
   http://www.asterisk.org/hello

asterisk-users mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-users


Re: [asterisk-users] offline builds - mp3 [patch]

2013-06-04 Thread Tzafrir Cohen
On Tue, Jun 04, 2013 at 06:44:43PM +0200, Daniel Pocock wrote:
 On 04/06/13 18:37, Tzafrir Cohen wrote:
  On Tue, Jun 04, 2013 at 12:49:35PM +0200, Daniel Pocock wrote:
 
 
  As mentioned in the thread about MP3, I found that the rpmbuild process
  demands network access, e.g. to access the mp3 code in SVN.
 
  Some people need to build on isolated networks though
 
  I've attached a patch that allows the MP3 code to be placed in /tmp
  before the build starts, then svn will not be used during the build.  If
  it finds /tmp/asterisk-contrib-mp3.tar.gz then it will be used instead
  of going to SVN
 
  I'm not sure if there are other build steps that access the network,
  this one was more obvious because I was trying to build on a fresh VM
  without any svn client
  
  I'm sure you're aware of:
  http://patch-tracker.debian.org/patch/series/view/asterisk/1:1.8.13.1~dfsg-3/mpglib
  
 
 The notes suggest that MP3 patent issues are a factor so I guessed
 that's why it is excluded from the tarball
 
 When building with rpmbuild the tarball is usually not unpacked
 manually, hence my own proposed patch looks in /tmp for the mp3 code -
 it could just as easily use your the patch from Debian as an input
 though, as long as it can be found in /tmp or some other predefined
 location.

How would you do that in a proper chrooted build?

The proper fix would be to applow to use a newer version of mpglib that
is included with some distributions.

-- 
   Tzafrir Cohen
icq#16849755  jabber:tzafrir.co...@xorcom.com
+972-50-7952406   mailto:tzafrir.co...@xorcom.com
http://www.xorcom.com  iax:gu...@local.xorcom.com/tzafrir

--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
New to Asterisk? Join us for a live introductory webinar every Thurs:
   http://www.asterisk.org/hello

asterisk-users mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-users


Re: [asterisk-users] offline builds - mp3 [patch]

2013-06-04 Thread Daniel Pocock
On 04/06/13 19:13, Tzafrir Cohen wrote:
 On Tue, Jun 04, 2013 at 06:44:43PM +0200, Daniel Pocock wrote:
 On 04/06/13 18:37, Tzafrir Cohen wrote:
 On Tue, Jun 04, 2013 at 12:49:35PM +0200, Daniel Pocock wrote:


 As mentioned in the thread about MP3, I found that the rpmbuild process
 demands network access, e.g. to access the mp3 code in SVN.

 Some people need to build on isolated networks though

 I've attached a patch that allows the MP3 code to be placed in /tmp
 before the build starts, then svn will not be used during the build.  If
 it finds /tmp/asterisk-contrib-mp3.tar.gz then it will be used instead
 of going to SVN

 I'm not sure if there are other build steps that access the network,
 this one was more obvious because I was trying to build on a fresh VM
 without any svn client

 I'm sure you're aware of:
 http://patch-tracker.debian.org/patch/series/view/asterisk/1:1.8.13.1~dfsg-3/mpglib


 The notes suggest that MP3 patent issues are a factor so I guessed
 that's why it is excluded from the tarball

 When building with rpmbuild the tarball is usually not unpacked
 manually, hence my own proposed patch looks in /tmp for the mp3 code -
 it could just as easily use your the patch from Debian as an input
 though, as long as it can be found in /tmp or some other predefined
 location.
 
 How would you do that in a proper chrooted build?
 
 The proper fix would be to applow to use a newer version of mpglib that
 is included with some distributions.
 

I'm not claiming that this was a proper fix - it is just a bare minimum
to allow offline builds with rpmbuild.  Although it has the feeling of a
hack, it doesn't prevent anybody implementing a more elegant solution in
future.

On the other hand, I was thinking about simply making up my own branch
of the code and a repackaged tarball and maybe even publishing some
convenient binary RPMs for everybody who wants to try this.  I realise
that asterisk-11.deb packages are a work in progress too, I didn't want
to put pressure on people to finish them, that's why I've just been
talking about the RPMs today.


--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
New to Asterisk? Join us for a live introductory webinar every Thurs:
   http://www.asterisk.org/hello

asterisk-users mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-users