sm-discuss AT lists.ibiblio.org
Subject: Public SourceMage Discussion List
List archive
Re: [SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works
- From: Paul Vinogradov <vin.public AT gmail.com>
- To: Ismael Luceno <ismael.luceno AT gmail.com>, sm-discuss AT lists.ibiblio.org
- Subject: Re: [SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works
- Date: Sat, 22 Mar 2014 02:02:46 -0400
OK, here we go.
В Sat, 22 Mar 2014 02:28:25 -0300
Ismael Luceno <ismael.luceno AT gmail.com> пишет:
> On Sat, 22 Mar 2014 01:22:37 -0400
> Paul Vinogradov <vin.public AT gmail.com> wrote:
> > What would we do with that kind of file? Theoretically.
> >
>
> No idea, but it should still not try to mangle the file name more than
> it should.
>
> > В Sat, 22 Mar 2014 01:39:30 -0300
> > Ismael Luceno <ismael.luceno AT gmail.com> пишет:
> >
> > > On Fri, Mar 21, 2014 at 5:20 PM, Paul Vinogradov
> > > <vin.public AT gmail.com> wrote:
> > > > В Fri, 21 Mar 2014 17:03:32 -0300
> > > > Ismael Luceno <ismael.luceno AT gmail.com> пишет:
> > > >
> > > >> On Thu, 20 Mar 2014 22:59:53 -0400
> > > >> Paul Vinogradov <vin.public AT gmail.com> wrote:
> > > >> > Hi all,
> > > >> >
> > > >> > Could someone with sorcery git access apply this patch? I'd
> > > >> > appreciate that.
> > > >> >
> > > >>
> > > >> Shouldn't it be "%.tar.*" instead of "%%.tar.*"?
> > > >
> > > > %% for great sureness. :)
> > >
> > > The problem is the following:
> > > ismael@pirotess:~$ V=file.tar.gz.tar.gz;echo ${V%.tar.*}
> > > file.tar.gz
> > > ismael@pirotess:~$ V=file.tar.gz.tar.gz;echo ${V%%.tar.*}
> > > file
> >
> >
>
--
Sincerely yours Paul Vinogradov
From 6f5769f18bfcfa41ef1a96f2d673b7df8264ca08 Mon Sep 17 00:00:00 2001
From: Pol Vinogradov <vin.public AT gmail.com>
Date: Thu, 20 Mar 2014 20:48:06 -0400
Subject: [PATCH] libsummon: repo archive no longer should be compressed by
bzip2 only, any compressor known to tar works
Signed-off-by: Pol Vinogradov <vin.public AT gmail.com>
---
ChangeLog | 4 ++++
var/lib/sorcery/modules/libsummon | 8 +++-----
2 files changed, 7 insertions(+), 5 deletions(-)
diff --git a/ChangeLog b/ChangeLog
index 96d9be6..0374ab1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2014-03-20 Pol Vinogradov <vin.public AT gmail.com>
+ * libsummon: repo archive no longer should be compressed by bzip2 only,
+ any compressor known to tar works
+
2013-04-24 Jaka Kranjc <lynxlynxlynx AT sourcemage.org>
* ensure sorcery tools are always first in PATH, to prevent conflicts
(delve is also in xapian-core), patch by Sukneet Basuta #506
diff --git a/var/lib/sorcery/modules/libsummon b/var/lib/sorcery/modules/libsummon
index 57cbc8a..d7da21e 100755
--- a/var/lib/sorcery/modules/libsummon
+++ b/var/lib/sorcery/modules/libsummon
@@ -181,8 +181,6 @@ function real_download_src() {
## Download the resource, handles file and tree discrepencies, if a tree is
## downloaded it is repackaged as the specified file for later use.
##
-## Only tar.bz2 files are acceptable caches for repackaged trees
-##
#-------------------------------------------------------------------------
function download_src_args() {
$STD_DEBUG
@@ -233,7 +231,7 @@ function download_src_args() {
# incorrect guess, set new_target appropriatly
if [[ "$guess_type" == file ]] ; then
- new_target="${target/.tar.bz2/}"
+ new_target="${target%.tar.*}"
fi
if ! test -d $summon_target; then
@@ -246,7 +244,7 @@ function download_src_args() {
mv -f $summon_target $new_target
fi &&
message "${MESSAGE_COLOR}Repackaging ${SPELL_COLOR}$target${DEFAULT_COLOR}"
- tar --remove-files -cjf $target $new_target &&
+ tar --remove-files -caf $target $new_target &&
rm -rf $new_target
elif [[ "$summon_type" == file ]] ; then
# there is no penalty for guessing tree when the result is a file
@@ -409,7 +407,7 @@ function unpack_for_update() {
local prefix=$(url_get_prefix $url_list)
if ! list_find "$hints" file &&
list_find "$hints" tree || list_find "$tree_prefixes" "$prefix" ; then
- _new_target="${target/.tar.bz2/}"
+ _new_target="${target%.tar.*}"
eval "$guess_type_ref=\"tree\""
eval "$new_target_ref=\"\$_new_target\""
--
1.9.0
Attachment:
signature.asc
Description: PGP signature
-
[SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works,
Paul Vinogradov, 03/20/2014
-
Re: [SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works,
Ismael Luceno, 03/21/2014
-
Message not available
-
Message not available
-
Message not available
-
Message not available
-
Re: [SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works,
Paul Vinogradov, 03/22/2014
- Re: [SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works, Ismael Luceno, 03/22/2014
- Re: [SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works, Jaka Kranjc, 03/23/2014
-
Re: [SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works,
Paul Vinogradov, 03/22/2014
-
Message not available
-
Message not available
-
Message not available
-
Message not available
-
Re: [SM-Discuss] [PATCH] libsummon: repo archive no longer should be compressed by bzip2 only, any compressor known to tar works,
Ismael Luceno, 03/21/2014
Archive powered by MHonArc 2.6.24.