Index: chrome/installer/linux/common/apt.include |
diff --git a/chrome/installer/linux/common/apt.include b/chrome/installer/linux/common/apt.include |
index cdc0c0ea5bc88f764394368d513cd8afc2edfae5..76c65a8a77018d73099ca87ca6b484f382546642 100644 |
--- a/chrome/installer/linux/common/apt.include |
+++ b/chrome/installer/linux/common/apt.include |
@@ -141,10 +141,14 @@ KEYDATA |
# Set variables for the locations of the apt sources lists. |
find_apt_sources() { |
+ # NB: These variables only *sometimes* include a trailing slash. (In |
+ # particular, in Ubuntu 16.10 / Debian 9, the default value *stopped* |
+ # including the trailing slash.) We have to join them with slashes, even |
+ # though that sometimes gives a double slash. |
Lei Zhang
2016/11/08 23:44:35
Does "${APTDIR%/}/" make it consistently 1 slash?
|
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')" |
} |
# Update the Google repository if it's not set correctly. |