proxy with uppercase HTTP causes postinst to fail

Bug #1072580 reported by Tobias
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
msttcorefonts (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Line 39 in postinst script fails because it doesn't match "HTTP://corp-proxy:8080/
    egrep -q 'https?://[[:alnum:]][-.[:alnum:]]+(:\d+)?/?' && \

The fix is trivial. Just add "-i" on the line:
    egrep -iq 'https?://[[:alnum:]][-.[:alnum:]]+(:\d+)?/?' && \

and the script works again

Related branches

Revision history for this message
Thijs Kinkhorst (kink) wrote :

Good suggestion, fixed in svn. Thanks!

Changed in msttcorefonts (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package msttcorefonts - 3.6ubuntu1

---------------
msttcorefonts (3.6ubuntu1) artful; urgency=medium

  * Merge from Debian Sid. Remaining changes:
    - Ubuntu preinst and postrm scripts have deltas.
    - Source the debconf confmodule before doing anything else in the preinst.
    - Move #DEBHELPER# substitution before preinst code that might exit.
    - Migrate to use an update-notifier hook, so that package data can be
      handled asynchronously and not cause install failures on network
      problems.
    - Add missing pre-depends for use of debconf in preinst.
    - Convert to dh(1).
    - Skip race between update-notifier and update-ms-fonts trying to delete
      the same files from the same locations.

msttcorefonts (3.6) unstable; urgency=medium

  * Add Turkish translation, thanks Mert Dirik (closes: #757868).
  * Update to policy 3.9.6, no changes.

msttcorefonts (3.5) unstable; urgency=low

  * Mark Multi-Arch: foreign, thanks Scott Ritchie, Daniel Hartwig
    (closes: #700684).
  * Switch to debhelper 9. Update for policy 3.9.4 (no changes).
  * Use maintscript support in dh_installdeb rather than writing out
    dpkg-maintscript-helper commands by hand. We now simply Pre-Depend on a
    new enough version of dpkg rather than using 'dpkg-maintscript-helper
    supports' guards, leading to more predictable behaviour on upgrades.
    Thanks Colin Watson (closes: #659724).
  * Proxy test failed when HTTP:// is given in uppercase. Thanks
    Tobias Olsson (LP: #1072580).
  * Show licence message when fonts actually installed (closes: #672259).

 -- Simon Quigley <email address hidden> Wed, 26 Jul 2017 22:02:46 -0500

Changed in msttcorefonts (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.