requestsync: prefer UBUEMAIL etc. environments over DEBEMAIL etc.

Bug #400133 reported by Martin-Éric Racine on 2009-07-16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Michael Bienia

Bug Description

Binary package hint: ubuntu-dev-tools

Currently, requestsync checks for Debian inherited environment variables DEBEMAIL, DEBSMTP, DEBSMTP_PORT, DEBSMTP_USER, DEBSMTP_PASS.

While this is not inherently wrong, it presumes that someone would be using the exact same e-mail environments for their Ubuntu work as for their Debian work, which doesn't always work:

While the GPG key might indeed often be the same one, the e-mail address would tend to be e.g. <email address hidden> to work on Debian and <email address hidden> to work on Ubuntu.

Right now, this differentiation is not possible, because Ubuntu tools like requestsync use the above existing Debian environment variables.

The suggestion would be to implement support for UBUEMAIL, UBUSMTP, UBUSMTP_PORT, UBUSMTP_USER, UBUSMTP_PASS. The strategy would be for UBU* variables to be preferred over DEB* variables and to only fall-back to DEB* variables if UBU* variables are not found.

Implementing this would greatly simplify the life of people like me who work in both distributions. :)

ProblemType: Bug
Architecture: i386
Date: Thu Jul 16 11:32:05 2009
DistroRelease: Ubuntu 9.10
Package: ubuntu-dev-tools 0.75
PackageArchitecture: all
 PATH=(custom, user)
ProcVersionSignature: Ubuntu 2.6.30-10.12-generic
SourcePackage: ubuntu-dev-tools
Uname: Linux 2.6.30-10-generic i586

Martin-Éric Racine (q-funk) wrote :
summary: - requestsync: please prefer UBUEMAIL etc. environments over DEBEMAIL etc.
+ requestsync: prefer UBUEMAIL etc. environments over DEBEMAIL etc.
Michael Bienia (geser) on 2009-08-22
Changed in ubuntu-dev-tools (Ubuntu):
status: New → In Progress
importance: Undecided → Wishlist
assignee: nobody → Michael Bienia (geser)
Michael Bienia (geser) wrote :

Fix committed to trunk (r484).

Changed in ubuntu-dev-tools (Ubuntu):
status: In Progress → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-dev-tools - 0.76

ubuntu-dev-tools (0.76) karmic; urgency=low

  [ Nicolas Valcárcel ]
  * mk-sbuild-lv:
    - Add check for built-in dm_snapshot (LP: #398414)

  [ Andreas Moog ]
  * update-maintainer:
    - Don't silently fail when Maintainer-Field contains a comment
      in brackets. (LP: #397144)
    - Don't add second XSBC-Original-Maintainer if Maintainer was set
      to Motu or Core-Dev.

  [ Michael Bienia ]
  * Drop python-launchpad-bugs from Depends.
  * Add python-lazr-restfulclient to Depends.
  * buildd: Add a --batch mode for batch retrying/rescoring of packages.
  * requestsync:
    - Use UBU* environment variables before the DEB* ones (lp: #400133)
    - Split requestsync into a "mail" module and a "lpapi" module and use
      the LP API only when --lp was used. In "mail" mode requestsync has
      to ask some more questions for parts it can't find out without LP API.
      (lp: #406659, #416955)

  [ Iain Lane ]
  * requestsync:
    - Guard some calls when -n is specified
    - Fetch changelog of specified version, not current version. If an
      experimenal upload happened after the unstable one we're syncing, this
      is considered to be current by p.d.o and we would get those changelog
      entries in the sync request
    - Remove trailing fullstop from sync bug title
  * suspicious-source: Add *.hs *.el *.css to whitelist

  [ Siegfried-Angel Gevatter Pujals ]
  * pbuilder-dist:
     - Expand "~" in PBUILDFOLDER to the user's home directory.
     - If there's a "etc/<distro>/apt.conf" file inside the build result
       directory, pass it to pbuilder as --aptconfdir. Thanks to Paul Novotny
       and Ryan Pavlik (LP: #363043).

  [ Luca Falavigna ]
  * Switch to python-support to ease initial import into Debian:
    - debian/control: build-depend on python-support instead of pycentral,
      also remove unneeded XB-Python-Version field from binary stanza.
    - debian/rules: set DEB_PYTHON_SYSTEM to pysupport.
    - ubuntu-dev-tools.preinst: remove stale pycentral files on upgrades.

  [ Nathan Handler ]
  * Add pull-revu-source and doc/pull-revu-source.1
  * Update debian/copyright to include pull-revu-source

 -- Nathan Handler <email address hidden> Sun, 30 Aug 2009 17:24:23 +0000

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

Other bug subscribers