[jitsi-dev] [jitsi/jitsi] Debian testing install failed in postinst. (#307)


#1

Hi,

There is an error in postinst script. There is no slash in the path (e.g. "etc/aptsources.list.d/jitsi.list").

Setting up jitsi (2.8.5426-1) ...
+ REPOCONFIG=deb http://download.jitsi.org/deb unstable/
+ SOURCES_PREAMBLE=### THIS FILE IS AUTOMATICALLY CONFIGURED ###
# You may comment out this entry, but any other modifications may be lost.\n
+ which apt-config
+ APT_CONFIG=/usr/bin/apt-config
+ install_key
+ which apt-key
+ APT_KEY=/usr/bin/apt-key
+ [ -x /usr/bin/apt-key ]
+ /usr/bin/apt-key add -
+ create_sources_lists
+ [ ! deb http://download.jitsi.org/deb unstable/ ]
+ find_apt_sources
+ apt_config_val Dir
+ APTVAR=Dir
+ [ -x /usr/bin/apt-config ]
+ /usr/bin/apt-config dump
+ sed -e /^Dir /!d -e s/^Dir "\(.*\)".*/\1/
+ APTDIR=/
+ apt_config_val Dir::Etc
+ APTVAR=Dir::Etc
+ [ -x /usr/bin/apt-config ]
+ /usr/bin/apt-config dump
+ sed -e /^Dir::Etc /!d -e s/^Dir::Etc "\(.*\)".*/\1/
+ APTETC=etc/apt
+ apt_config_val Dir::Etc::sourcelist
+ APTVAR=Dir::Etc::sourcelist
+ [ -x /usr/bin/apt-config ]
+ /usr/bin/apt-config dump
+ sed -e /^Dir::Etc::sourcelist /!d -e s/^Dir::Etc::sourcelist "\(.*\)".*/\1/
+ APT_SOURCES=/etc/aptsources.list
+ apt_config_val Dir::Etc::sourceparts
+ APTVAR=Dir::Etc::sourceparts
+ [ -x /usr/bin/apt-config ]
+ /usr/bin/apt-config dump
+ sed -e /^Dir::Etc::sourceparts /!d -e s/^Dir::Etc::sourceparts "\(.*\)".*/\1/
+ APT_SOURCESDIR=/etc/aptsources.list.d
+ SOURCELIST=/etc/aptsources.list.d/jitsi.list
+ grep ^[[:space:]#]*\bdeb http://download.jitsi.org/deb unstable/\b /etc/aptsources.list.d/jitsi.list
+ REPOMATCH=
+ [ -r /etc/aptsources.list.d/jitsi.list ]
+ [ -d /etc/aptsources.list.d ]
+ return 2
dpkg: error processing package jitsi (--configure):

Adding slash fixes the problem:

--- /var/lib/dpkg/info/jitsi.postinst.orig   2016-12-08 13:54:54.298328738 +0100
+++ /var/lib/dpkg/info/jitsi.postinst 2016-12-08 13:54:35.746996376 +0100
@@ -65,8 +65,8 @@
 find_apt_sources() {
   APTDIR=$(apt_config_val Dir)
   APTETC=$(apt_config_val 'Dir::Etc')
-  APT_SOURCES="$APTDIR$APTETC$(apt_config_val 'Dir::Etc::sourcelist')"
-  APT_SOURCESDIR="$APTDIR$APTETC$(apt_config_val 'Dir::Etc::sourceparts')"
+  APT_SOURCES="$APTDIR$APTETC/$(apt_config_val 'Dir::Etc::sourcelist')"
+  APT_SOURCESDIR="$APTDIR$APTETC/$(apt_config_val 'Dir::Etc::sourceparts')"
 }
 
 # Add the Jitsi repository to the apt sources.

OS: Debian stretch/sid (amd64).

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/307


#2

@damencho Can you please have a look? I don't even know where the `postinst` file is coming from.

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/307#issuecomment-266874408


#3

Hi, this code for adding apt-sources had been removed in latest nightlies (https://github.com/jitsi/jitsi/commit/5d9f0844914bb00732b5cb82670b422230499ccb). And this should no longer be a problem. The package now depends on jitsi-archive-keyring, which will install the file directly and will use the new debian repo's. Not sure how this stop working, but the problem will be fixed once we update the stable package.

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/307#issuecomment-267636451


#4

Closed #307.

···

--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jitsi/jitsi/issues/307#event-896934703