Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Debian testing install failed in postinst. #307

Closed
rysson opened this issue Dec 8, 2016 · 2 comments
Closed

Debian testing install failed in postinst. #307

rysson opened this issue Dec 8, 2016 · 2 comments

Comments

@rysson
Copy link

rysson commented Dec 8, 2016

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).

@ibauersachs
Copy link
Member

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

@damencho
Copy link
Member

Hi, this code for adding apt-sources had been removed in latest nightlies (5d9f084). 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants