requestsync can't handle third parameter (basever) but advertises it in usage

Bug #119313 reported by Kjell Braden
2
Affects Status Importance Assigned to Milestone
devscripts (Ubuntu)
Fix Released
Low
Adrien Cunin

Bug Description

Binary package hint: devscripts

$ requestsync
Usage: requestsync [-n|-s] <source package> <target release> [basever]

In some cases, the base version (fork point from Debian) cannot be determined
automatically, and you'll get a complete Debian changelog. Specify the correct
base version in that case.
$ requestsync empathy gutsy 0.5-1
Traceback (most recent call last):
  File "/usr/bin/requestsync", line 90, in <module>
    (srcpkg, release) = args
ValueError: too many values to unpack
$ LC_ALL=c apt-cache policy devscripts
devscripts:
  Installed: 2.10.4ubuntu6
  Candidate: 2.10.4ubuntu6
  Version table:
 *** 2.10.4ubuntu6 0
        500 http://archive.ubuntu.com gutsy/main Packages
        100 /var/lib/dpkg/status

Tags: patch

Related branches

Revision history for this message
Kjell Braden (afflux) wrote :

This patch fixes the issue for me.

Revision history for this message
Kjell Braden (afflux) wrote :

The heat constrains my brain.

Changed in devscripts:
status: Unconfirmed → Confirmed
Adrien Cunin (adri2000)
Changed in devscripts:
assignee: nobody → adri2000
importance: Undecided → Low
Revision history for this message
Adrien Cunin (adri2000) wrote :

The fix will be uploaded soon, thanks.

Changed in devscripts:
status: Confirmed → Fix Committed
Revision history for this message
Steve Kowalik (stevenk) wrote :
Download full text (3.4 KiB)

devscripts (2.10.5ubuntu1) gutsy; urgency=low

  * Merge from Debian unstable
  * Remaining Ubuntu changes:
    - requestsync:
      + New script.
      + ask for explanation why Ubuntu changes can be dropped (patch from
        Adrien Cunin)
      + use gpg2 for signing if available
      + General cleanup, mostly to stop mixing tabs and spaces.
      + Add a -n option, to show that this is a new source package according
        to Ubuntu.
      + Add a -s option, that will subscribe the relevant sponsor group
        instead of ubuntu-archive.
      + Look up the Debian component, and support downloading changelog
        entries from components other than main.
      + Print a meaningful error rather than throwing an exception if
        apt-cache madison contains nothing.
      + Don't confirm the bug when -s (sponsorship) is passed
      + Giving base version as a third argument works again, thanks Kjell
        Braden. (LP: #119313)
    - debchange: add --security option, which acts like --nmu, but adds
      security update template instead of NMU template to changelog.
    - debuild:
      + Add some heuristic checks about the Ubuntu merging policy. Issue a
        warning if a merge does not use -v or describe the remaining Ubuntu
        changes.
      + Keep DISPLAY, XAUTHORITY and GNOME_KEYRING_SOCKET environment
        variables around for safe keeping before we destroy the environment,
        and re-set them before running debsign. (LP: #78165)
  * Ubuntu changes dropped:
    - debchange:
      + Add support for Ubuntu versioning.
      + Allow UNRELEASED in Ubuntu distro check.
      + Accept Ubuntu distribution "gutsy" and use it by default on Ubuntu.

devscripts (2.10.5) unstable; urgency=low

  [ Luk Claes ]
  * debc: Make reporting on a single deb work again (Closes: #419487).

  [ Adam D. Barratt ]
  * debchange: Add gutsy (gibbon) and UNRELEASED to the list of recognised
    Ubuntu distributions
  * rc-alert
    + Update HTML parsing to support the new structure of the RC bug page
    + The "I" tag can now also mean "lenny-ingore"
    + Add "[E]xperimental" to the distribution list, and remove "X"
      ("not in testing")
  * bts
    + Allow the use of SMTP instead of sendmail. Thanks to Josh Tripplet for
      the patch (Closes: #420847)
    + Fix syntax error when reverting to default SENDMAILCMD
    + When cleaning up a bug's cache entries, also remove {raw,status}.mbox
    + Fix pod formatting so that the description of the "fixed" command
      actually appears in the manpage.
    + Add an option to not cache resolved bugs (Closes: #340259, #378342)
    + Fix URL parsing to account for recent HTML changes (Closes: #429236)
    + Always download version graphs rather than the new "options page"
    + Fix attachment parsing code to handle "filename="
  * mass-bug: Fix syntax error when reverting to default SENDMAILCMD
  * dd-list: Treat teams.debian.net addresses as mailing lists rather
    than individuals
  * tagpending: Update license statement to GPL >= 2
  * uscan: Ensure --dehs generates well formed XML (Closes: #426996)

  [ Martin Zobel-Helas ]
  * debchange: Add volatile to the list of recognised Debian dist...

Read more...

Changed in devscripts:
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.