[SRM] dpkg 1.15.8.11 for squeeze

[Guillem Jover]

Hi!

We've fixed several regressions on the 1.15.x series, and I'd like to
upload a new 1.15.8.11 release targetting squeeze. I'm attaching the
interesting git commits (omitted translations and a revert). Maybe
there will still be additional translation updates before the upload
though. Here's the changelog:


dpkg (1.15.8.11) stable; urgency=low

  [ Guillem Jover ]
  * Do not segfault on “dpkg -i --no-act”.
  * Add missing semicolon to the vsnprintf() compat declaration.
Thanks to Robert Millan. Closes: #612203
  * Fix typo in «dpkg-name --overwrite» argument parsing so that it actually
works at all. Thanks to Ivan Gagis . LP: #728708
  * Fix dpkg-split to not corrupt binary part metadata when generating the
split packages on 32-bit systems.

  [ Raphaël Hertzog ]
  * Fix a regression in dpkg-divert where using --rename led to a failure when
the rename implies crossing file systems. Thanks to Durk Strooisma for
spotting it.

  [ Updated dpkg translations ]
  * German (Sven Joachim).

  [ Updated man page translations ]
  * German (Helge Kreutzmann). Minor fixe(s).

 -- Guillem Jover   Sun, 17 Apr 2011 07:27:13 +0200


The git branch for squeeze can be found at:

  

Would such upload be fine?

thanks,
guillem
commit 3be9001824ac3eb1b7e2cd6bfee67eb35166dab6
Author: Guillem Jover 
Date:   Fri Apr 15 06:02:34 2011 +0200

dpkg-split: Do not corrupt binary part metadata on 32-bit systems

On 32-bit systems the format string used to generate the metadata
archive member of a split package specifies a 32-bit value to be
retrieved from va_arg, but the variable passed is a 64-bit one, which
messes with the layout of the stack, producing garbage on output.

Regression introduced in commit f2115151c19ff37b305296c23225807b0832086b.

diff --git a/debian/changelog b/debian/changelog
index d9d01ca..21371f0 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -7,6 +7,8 @@ dpkg (1.15.8.11) UNRELEASED; urgency=low
 Thanks to Robert Millan. Closes: #612203
   * Fix typo in «dpkg-name --overwrite» argument parsing so that it actually
 works at all. Thanks to Ivan Gagis . LP: #728708
+  * Fix dpkg-split to not corrupt binary part metadata when generating the
+split packages on 32-bit systems.
 
   [ Raphaël Hertzog ]
   * Fix a regression in dpkg-divert where using --rename led to a failure when
diff --git a/dpkg-split/split.c b/dpkg-split/split.c
index 46262f1..324c885 100644
--- a/dpkg-split/split.c
+++ b/dpkg-split/split.c
@@ -206,7 +206,7 @@ mksplit(const char *file_src, const char *prefix, size_t partsize,
 		/* Write the debian-split part. */
 		varbufprintf(&partmagic, "%s\n%s\n%s\n%s\n%zu\n%zu\n%d/%d\n",
 		 SPLITVERSION, package, version, hash,
-		 st.st_size, partsize, curpart, nparts);
+		 (size_t)st.st_size, partsize, curpart, nparts);
 		dpkg_ar_member_put_mem(file_dst.buf, fd_dst, PARTMAGIC,
 		   partmagic.buf, partmagic.used);
 		varbufreset(&partmagic);

commit de3dcef863d389af305009bbd12862c537be99e4
Author: Ivan Gagis 
Date:   Wed Mar 9 05:18:09 2011 +0100

dpkg-name: Fix typo in --overwrite argument parsing

Cherry picked from commit 8a910bc7aadd0525fe24b14c8d6249f77bba15b7.

This fixes the option so that it actually works at all.

LP: #728708

Signed-off-by: Guillem Jover 

diff --git a/debian/changelog b/debian/changelog
index b1821d5..d9d01ca 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -5,6 +5,8 @@ dpkg (1.15.8.11) UNRELEASED; urgency=low
   * Do not segfault on “dpkg -i --no-act”.
   * Add missing semicolon to the vsnprintf() compat declaration.
 Thanks to Robert Millan. Closes: #612203
+  * Fix typo in «dpkg-name --overwrite» argument parsing so that it actually
+works at all. Thanks to Ivan Gagis . LP: #728708
 
   [ Raphaël Hertzog ]
   * Fix a regression in dpkg-divert where using --rename led to a failure when
diff --git a/scripts/dpkg-name.pl b/scripts/dpkg-name.pl
index 6745ac1..30c1aa6 100755
--- a/scripts/dpkg-name.pl
+++ b/scripts/dpkg-name.pl
@@ -233,7 +233,7 @@ while (@ARGV) {
 $options{destdir} = shift(@ARGV);
 }
 } elsif (m/^-o|--overwrite$/) {
-$options{overwite} = 1;
+$options{overwrite} = 1;
 } elsif (m/^-k|--symlink$/) {
 $options{symlink} = 1;
 } elsif (m/^-a|--no-architecture$/) {

commit 9387f083bec3d94312e1f80162a29d73809feded
Author: Raphaël Hertzog 
Date:   Thu Mar 17 14:35:53 2011 +0100

dpkg-divert: fix return value of rename_mv

This is a regression introduced by the C rewrite of dpkg-divert
(commit 02b12d75f1eff7c184fafb5a663a0421e9a645ea) that made it unable to
rename files when that rename implies a copy on another file system.

The code has provision to do a copy when a simple rename is not enough
but the associated function erroneously re

Bug#622947: per-maintainer insights into migrations and transitions

[Paul Wise]

On Sat, 2011-04-16 at 13:33 +0200, Mehdi Dogguy wrote:

> grep-excuses takes into account DEBFULLNAME, if no packages or maintainer
> are given on the command line. Isn't that what you want?

That doesn't appear to work with -w nor with an email address since
update_excuses.html.gz doesn't contain any. It would be nice if I could
specify a command-line option so I could easily look up team packages.

In any case I'd prefer a web page to look at than a command to run.

-- 
bye,
pabs

http://wiki.debian.org/PaulWise


signature.asc
Description: This is a digitally signed message part

Bug#621101: transition: db (db4,6, db4.7, db4.8)

[Ricardo Mones]

On Sat, 16 Apr 2011 22:30:13 +0200
Mehdi Dogguy  wrote:

> >Just curious, shouldn't libetpan be appearing on that list?
> >IIRC it build-depends on libdb-dev.
> >  
> 
> Is *is* there. Untick "hide fully (re-)built packages" to see it.

  Oops, you're absolutely right, sorry for the noise ;)
-- 
 Ricardo Mones
 http://people.debian.org/~mones
 «Do not sleep in a eucalyptus tree tonight.»


signature.asc
Description: PGP signature

Bug#621101: transition: db (db4,6, db4.7, db4.8)

[Mehdi Dogguy]

On 04/16/2011 08:31 PM, Ricardo Mones wrote:

On Thu, 14 Apr 2011 15:52:45 +0200
Mehdi Dogguy  wrote:


On 06/04/2011 16:43, Ondřej Surý wrote:


I would like to coordinate reduction of BDB packages since I took the
unhappy job (as I could expect) to maintain BDB in Debian after Clint
have orphaned them.



I've added a page for libdb on the transition tracker so that we follow
the status of this transition. It's available at:

http://release.debian.org/transitions/html/libdb.html


   Just curious, shouldn't libetpan be appearing on that list?
   IIRC it build-depends on libdb-dev.



Is *is* there. Untick "hide fully (re-)built packages" to see it.

Regards,

--
Mehdi Dogguy مهدي الدڤي
http://dogguy.org/



--
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4da9fc55.1030...@dogguy.org

Re: Fwd: Bug#616482: strongswan-ikev1: virtual ips not released if xauth name does not match id

[René Mayrhofer]

Am Freitag 15 April 2011, um 19:33:05 schrieb Adam D. Barratt:
> On Wed, 2011-04-06 at 21:46 +0200, René Mayrhofer wrote:
> > I have now integrated the cherry-picked upstream patch into my
> > strongswan-sqeeze branch at the alioth git repository
> > (ssh://alioth.debian.org/git/pkg-swan/strongswan.git). As mentioned in
> > the bug report, it applies cleanly and is an isolated fix for a bug in
> > version 4.4.1 that impacts some clients.
> 
> It looks like we managed to miss this when it was originally sent;
> apologies for that.  However, a lack of response should only be treated
> as that, not as an implicit ack for an upload.

I will remember that for future uploads.
 
> Why have the configure options in debian/rules been modified, with no
> mention of this in the changelog?  So far as I can see, --enable-pkcs11,
> --enable-eap-tls, --enable-eap-ttls and --enable-led have been added.
> The addition of "--enable-nat-transport" /is/ mentioned in the
> changelog, but was not mentioned in your mail to -release.

Sorry about that, I only mentioned the last change that actually triggered the 
desire to get it into proposed-updates for squeeze. The other changes were made 
in my squeeze branch of strongswan that I use in production for the Gibraltar 
firewall. Enabling the additional modules was required in some cases but should 
not lead to any regressions, as these modules need to be enabled explicitly in 
the config file to be used. With default config, that means no changes to the 
previous version in squeeze.
 
> There are also several other changes related to the removal and
> re-enabling under certain circumstances of the Network Manager support.
> The associated comments indicate that this change was intended to ease
> backports, so I'm not sure why this is being included in a stable
> update; again, these changes are not mentioned in the changelog.
> 
> Furthermore, the n-m changes are actually buggy:
> 
> +  CONFIGUREARS += --enable-nm
> 
> That should presumably be CONFIGUREAR*G*S.

Thanks for spotting that, I have fixed it in my git branch. The change was 
indeed made for the same in-production packages we use, as the older version of 
Gibraltar firewall is based on Lenny and we backport important packages such as 
strongswan.

The current version has been tested by us on many fireewalls and by the 
original bug reporter without any regressions being found.

Would you prefer that I upload a new package with the fix to debian/rules 
network-manager handling with a new version or just again with the same one?

best regards,
Rene



signature.asc
Description: This is a digitally signed message part.

NEW changes in proposedupdates

[Debian FTP Masters]

Processing changes file: josm_0.0.svn3376-1+squeeze1_i386.changes
  ACCEPT
Processing changes file: softhsm_1.1.4-4+squeeze1_amd64.changes
  ACCEPT


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/e1qbbha-0004lz...@franck.debian.org

Bug#621101: transition: db (db4,6, db4.7, db4.8)

[Ricardo Mones]

On Thu, 14 Apr 2011 15:52:45 +0200
Mehdi Dogguy  wrote:

> On 06/04/2011 16:43, Ondřej Surý wrote:
> >
> > I would like to coordinate reduction of BDB packages since I took the
> > unhappy job (as I could expect) to maintain BDB in Debian after Clint
> > have orphaned them.
> >
> 
> I've added a page for libdb on the transition tracker so that we follow
> the status of this transition. It's available at:
> 
>   http://release.debian.org/transitions/html/libdb.html

  Just curious, shouldn't libetpan be appearing on that list?
  IIRC it build-depends on libdb-dev.

  regards,
-- 
 Ricardo Mones
 http://people.debian.org/~mones
 «If you stand on your head, you will get footprints in your hair.»


signature.asc
Description: PGP signature

Bug#623014: gcc-defaults: please default to gcc-4.5 on mips and mipsel

[Aurelien Jarno]

Package: gcc-defaults
Version: 1.102
Severity: wishlist

Now that gcc-4.5 seems to be in a good shape, I think it's time to 
make it the default on mips and mipsel. Note that it introduces a few
changes:
- The default ABI in gcc-4.5 is MIPS II instead of MIPS I. It means that
  it's not necessary anymore to use '.set mips2' pseudo ops in assembly 
  code to get access to the atomic instructions (ll, sc, lld, scd).
  gcc-4.5 also doesn't emit any more delay slot after load/store
  instructions, which slightly reduce the size of the generated 
  binaries. The drawback is that MIPS3000 CPUs are not supported 
  anymore. It should not be a big deal, especially given we don't 
  provide kernels for these CPUs for years. 
- gcc-4.5 enables PLT and copy relocation extensions of MIPS ABI by
  default (-mplt), which produces faster binaries.

-- System Information:
Debian Release: wheezy/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: mips (mips64)

Kernel: Linux 2.6.32-5-5kc-malta
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/bash



-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/20110416161957.16467.81305.report...@volta.aurel32.net

Ispell transition

[Mehdi Dogguy]

Hi.

Sorry if this topic has been raised already somewhere else, but didn't
find any details about your transition.

ispell 3.3.02-4, currently staying in sid, Breaks several packages
providing an ispell-dictionary. While I'm sure you have good reasons to
do so, I do wonder if you already talked to other maintainers so that
they start updating their packages.

It seems that the change that's causing most troubles right now is:

 + add versioned `Breaks' for all packages providing
   ispell-dictionary that are not architecture `all';

which looks like:

Breaks: iamerican (<= 3.1.20.0-9), ibritish (<= 3.1.20.0-9), ibulgarian
(<= 3.0-12), iczech (<= 20040229-5), idanish (<= 1.6.25-1), iestonian
(<= 1:20030606-12.1), ifinnish (<= 0.7-17.3), ifinnish-large (<=
0.7-17.3), ifrench (<= 1.4-25), ifrench-gut (<= 1:1.0-27), igaelic (<=
0.50-7), ihungarian (<= 0.99.4-2), iirish (<= 2.0-20), iitalian (<=
1:2.3-2), imanx (<= 0.50-8), inorwegian (<= 2.0.10-3.2), iogerman (<=
1:2-26), ipolish (<= 20100612-1), irussian (<= 0.99g5-8.1), iswedish (<=
1.4.5-2), iukrainian (<= 1.6.0-1)

A quick investigation gives us the list below (list of packages that
need to be updated). Did you try to contact those maintainers, explain
the plan and ask them to update their package? ... and file bugreports?

As long as this list of packages is not reduced to zero, ispell won't be
able to migrate to testing.

Jeremiah C. Foster 
   swedish

Tollef Fog Heen 
   norwegian

Kalle Kivimaa 
   ispell-fi

Francesco Paolo Lovergine 
   iitalian

Anton Martchukov 
   rus-ispell (U)

Martin-Éric Racine 
   ispell-et
   rus-ispell

Jonas Smedegaard 
   dsdo

Petr Čech 
   ispell-czech

Regards,

--
Mehdi Dogguy مهدي الدڤي
http://dogguy.org/


--
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4da9bd51.20...@dogguy.org

Processed: tagging 618871

[Debian Bug Tracking System]

Processing commands for cont...@bugs.debian.org:

> tags 618871 + pending
Bug #618871 [release.debian.org] transition: ocaml
Added tag(s) pending.
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
618871: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=618871
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/handler.s.c.130296775325688.transcr...@bugs.debian.org

Bug#622947: per-maintainer insights into migrations and transitions

[Mehdi Dogguy]

On 04/16/2011 02:51 PM, Adam D. Barratt wrote:

On Sat, 2011-04-16 at 13:33 +0200, Mehdi Dogguy wrote:

On 04/16/2011 06:49 AM, Paul Wise wrote:

It would be nice to be able to easily find out:

* which of the packages I am responsible for haven't yet
migrated to testing after the 10/5/2 day period and why


grep-excuses takes into account DEBFULLNAME, if no packages or
maintainer are given on the command line. Isn't that what you
want?

The manpage mentions GREP_EXCUSES_MAINTAINER, but it's not used in
the script (Actually, it's set to an empty string and not read from
env).


Errr... yes, it's used in the script.  It's not read from the
environment, because it's not an environment variable, as
grep-excuses(1) explicitly says.  It *is* read from the
configuration file and used to initially populate $string.



Right. I should read the manpage next time, instead of capturing some
parts of it :)

Anyway... I'm still waiting for Pabs's comment :p

Regards,

--
Mehdi Dogguy مهدي الدڤي
http://dogguy.org/



--
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4da9962a.3080...@dogguy.org

Processed: Re: Bug#620975: transition: python-apt

[Debian Bug Tracking System]

Processing commands for cont...@bugs.debian.org:

> block 620975 with 622279
Bug #620975 [release.debian.org] transition: python-apt
Was blocked by: 617272 572072 604544 572350 572087 572059
Added blocking bug(s) of 620975: 622279
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
620975: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=620975
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/handler.s.c.130295861328700.transcr...@bugs.debian.org

Bug#620975: transition: python-apt

[Julian Andres Klode]

block 620975 with 622279
thanks

On Di, 2011-04-05 at 16:48 +0200, Julian Andres Klode wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian@packages.debian.org
> Usertags: transition
> 
> As I wrote some time ago, the APT team would like to disable
> the old python-apt API for Python 2.X now.
> 
> Preparation for this transition started more than a year ago,
> and most packages are now fixed. The transition can be seen in
> [1].
> 
> Some time ago, we uploaded 0.7.100+newapi with old API disabled
> to experimental in order to allow further testing. We now uploaded
> 0.8.0~exp1 which does this again.
> 
> After the Python transition(s) and once we finished our
> changes for the 0.8 release (1,2,3 weeks), we'd like to
> upload a 0.8 release to unstable.
The python-apt transition is not urgent, and as I wrote, should be done
after the Python transitions, so let's block it by the python-defaults
transition. And having more time for python-apt 0.8's multi-arch stuff
may be a good idea as well, we're still missing a few properties here
and there.

Of course, if you want to do this transition first, change the blocking
direction.

> Packages remaining to be fixed/affected by this transition:
>   - python-debian (needs update)
Fixed

>   - debdelta (leaf package)
>   - dogtail (leaf package, removefromtesting)
>   - computer-janitor (leaf package, removefromtesting)
No status change

>   - software-properties (needs update)
Fixed

There's also a new bug with update-notifier, although I haven't reported
it yet. update-notifier currently does not byte-compile it's python
script or use python in its build, so it's not part of the python
transition, and can be updated while the transition is undergoing.


-- 
Julian Andres Klode  - Debian Developer, Ubuntu Member

See http://wiki.debian.org/JulianAndresKlode and http://jak-linux.org/.





-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/1302958598.5936.14.camel@jak-thinkpad

Bug#622947: per-maintainer insights into migrations and transitions

[Adam D. Barratt]

On Sat, 2011-04-16 at 13:33 +0200, Mehdi Dogguy wrote:
> On 04/16/2011 06:49 AM, Paul Wise wrote:
> > It would be nice to be able to easily find out:
> >
> >* which of the packages I am responsible for haven't yet migrated
> >  to testing after the 10/5/2 day period and why
> 
> grep-excuses takes into account DEBFULLNAME, if no packages or maintainer
> are given on the command line. Isn't that what you want?
> 
> The manpage mentions GREP_EXCUSES_MAINTAINER, but it's not used in the 
> script
> (Actually, it's set to an empty string and not read from env).

Errr... yes, it's used in the script.  It's not read from the
environment, because it's not an environment variable, as
grep-excuses(1) explicitly says.  It *is* read from the configuration
file and used to initially populate $string.



Regards,

Adam




-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/1302958273.3388.841.ca...@hathi.jungle.funky-badger.org

Processed: block 622279 with 622912

[Debian Bug Tracking System]

Processing commands for cont...@bugs.debian.org:

> block 622279 with 622912
Bug #622279 [release.debian.org] transition: python-defaults
Was blocked by: 622154 616364 622001 621993 618094 606681 622976 621932
Added blocking bug(s) of 622279: 622966, 622978, and 622912
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
622279: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622279
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/handler.s.c.130295774524922.transcr...@bugs.debian.org

Processed: unblock 622279 with 622978

[Debian Bug Tracking System]

Processing commands for cont...@bugs.debian.org:

> unblock 622279 with 622978
Bug #622279 [release.debian.org] transition: python-defaults
Was blocked by: 622154 616364 622001 621993 618094 622978 606681 621932 622976
Removed blocking bug(s) of 622279: 622966, 622978, and 622912
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
622279: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622279
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/handler.s.c.130295728023216.transcr...@bugs.debian.org

Bug#622947: per-maintainer insights into migrations and transitions

[Mehdi Dogguy]

On 04/16/2011 06:49 AM, Paul Wise wrote:

Package: release.debian.org
Severity: wishlist

It would be nice to be able to easily find out:

   * which of the packages I am responsible for haven't yet migrated
 to testing after the 10/5/2 day period and why


grep-excuses takes into account DEBFULLNAME, if no packages or maintainer
are given on the command line. Isn't that what you want?

The manpage mentions GREP_EXCUSES_MAINTAINER, but it's not used in the 
script

(Actually, it's set to an empty string and not read from env).


   * which of the packages I am responsible for are involved in
 current transitions
   * which of the packages I am responsible for are involved in a
 specific transition



Regards,

--
Mehdi Dogguy مهدي الدڤي
http://dogguy.org/



--
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: http://lists.debian.org/4da97e8d.5020...@dogguy.org

Processed: block 622279 with 622976 622978

[Debian Bug Tracking System]

Processing commands for cont...@bugs.debian.org:

> block 622279 with 622976 622978
Bug #622279 [release.debian.org] transition: python-defaults
Was blocked by: 622154 616364 618094 606681 622001 621932 621993
Added blocking bug(s) of 622279: 622978 and 622976
> thanks
Stopping processing here.

Please contact me if you need assistance.
-- 
622279: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=622279
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems


-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/handler.s.c.13029513092928.transcr...@bugs.debian.org

Bug#622973: pu: package josm/0.0.svn3376-1+squeeze1

[David Paleino]

On Sat, 16 Apr 2011 11:32:42 +0100, Adam D. Barratt wrote:

> On Sat, 2011-04-16 at 11:55 +0200, David Paleino wrote:
> > I'm writing to you to ask for a possible p-u upload for JOSM.
> > 
> > OpenStreetMap is pursuing a process to change the license of its data.
> > Tomorrow it will enter Phase III, i.e. a contributor has to explicitely
> > accept or decline the new license [0], otherwise he'll get a 403 error [1].
> 
> If this is phase 3, presumably this has been known about for a while?

The date wasn't known up to two days ago; when it was announced (see link [0]
in my mail).

> If so, it would have been nicer if the patch could have been included in
> the version which we shipped in Squeeze.  (I realise this was only just
> fixed upstream, looking at the links you provided; thanks for those).

Heh :)
It's really an easy patch, but the problem really is that it was announced
suddenly, and with little pre-notice.

> > The suggested fix [2] was to improve the error message, including the
> > response body. Upstream did this [3], and released a new snapshot (4021,
> > currently in sid).
> 
> Assuming the resulting package has been tested in a Squeeze environment,
> please feel free to upload including (just) that patch.

Thanks,
David

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174


signature.asc
Description: PGP signature

Bug#622973: pu: package josm/0.0.svn3376-1+squeeze1

[Adam D. Barratt]

On Sat, 2011-04-16 at 11:55 +0200, David Paleino wrote:
> I'm writing to you to ask for a possible p-u upload for JOSM.
> 
> OpenStreetMap is pursuing a process to change the license of its data.
> Tomorrow it will enter Phase III, i.e. a contributor has to explicitely accept
> or decline the new license [0], otherwise he'll get a 403 error [1].

If this is phase 3, presumably this has been known about for a while?
If so, it would have been nicer if the patch could have been included in
the version which we shipped in Squeeze.  (I realise this was only just
fixed upstream, looking at the links you provided; thanks for those).

> The suggested fix [2] was to improve the error message, including the response
> body. Upstream did this [3], and released a new snapshot (4021, currently in
> sid).

Assuming the resulting package has been tested in a Squeeze environment,
please feel free to upload including (just) that patch.

> [0]: http://lists.openstreetmap.org/pipermail/talk/2011-April/057649.html
> [1]: http://lists.openstreetmap.org/pipermail/josm-dev/2011-April/005504.html
> [2]: http://lists.openstreetmap.org/pipermail/josm-dev/2011-April/005508.html
> [3]:
> https://josm.openstreetmap.de/changeset?reponame=josm&new=4020%40%2F&old=4019%40%2F

Regards,

Adam




-- 
To UNSUBSCRIBE, email to debian-release-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org
Archive: 
http://lists.debian.org/1302949962.3388.281.ca...@hathi.jungle.funky-badger.org

Bug#622973: pu: package josm/0.0.svn3376-1+squeeze1

[David Paleino]

Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: pu

Dear Release Team,
I'm writing to you to ask for a possible p-u upload for JOSM.

OpenStreetMap is pursuing a process to change the license of its data.
Tomorrow it will enter Phase III, i.e. a contributor has to explicitely accept
or decline the new license [0], otherwise he'll get a 403 error [1].

The suggested fix [2] was to improve the error message, including the response
body. Upstream did this [3], and released a new snapshot (4021, currently in
sid).

I'm attaching the backported patch.

Thanks,
David

[0]: http://lists.openstreetmap.org/pipermail/talk/2011-April/057649.html
[1]: http://lists.openstreetmap.org/pipermail/josm-dev/2011-April/005504.html
[2]: http://lists.openstreetmap.org/pipermail/josm-dev/2011-April/005508.html
[3]:
https://josm.openstreetmap.de/changeset?reponame=josm&new=4020%40%2F&old=4019%40%2F

-- 
 . ''`.   Debian developer | http://wiki.debian.org/DavidPaleino
 : :'  : Linuxer #334216 --|-- http://www.hanskalabs.net/
 `. `'`  GPG: 1392B174 | http://deb.li/dapal
   `-   2BAB C625 4E66 E7B8 450A C3E1 E6AA 9017 1392 B174
From: David Paleino 
Subject: backport fixed failed authorisation dialog from 4021
Origin: vendor
Forwarded: not-needed

---
 src/org/openstreetmap/josm/gui/ExceptionDialogUtil.java |3 +-
 src/org/openstreetmap/josm/tools/ExceptionUtil.java |   24 
 2 files changed, 26 insertions(+), 1 deletion(-)

--- josm.orig/src/org/openstreetmap/josm/gui/ExceptionDialogUtil.java
+++ josm/src/org/openstreetmap/josm/gui/ExceptionDialogUtil.java
@@ -271,9 +271,10 @@ public class ExceptionDialogUtil {
  * @param e the exception
  */
 public static void explainAuthorizationFailed(OsmApiException e) {
+// Fixme: add special handling that calls ExceptionUtil.explainFailedOAuthAuthorisation(e)
 HelpAwareOptionPane.showOptionDialog(
 Main.parent,
-ExceptionUtil.explainFailedOAuthAuthorisation(e),
+ExceptionUtil.explainFailedAuthorisation(e),
 tr("Authorisation Failed"),
 JOptionPane.ERROR_MESSAGE,
 ht("/ErrorMessages#AuthenticationFailed")
--- josm.orig/src/org/openstreetmap/josm/tools/ExceptionUtil.java
+++ josm/src/org/openstreetmap/josm/tools/ExceptionUtil.java
@@ -128,6 +128,30 @@ public class ExceptionUtil {
 );
 }
 
+   public static String explainFailedAuthorisation(OsmApiException e) {
+e.printStackTrace();
+String header = e.getErrorHeader();
+String body = e.getErrorBody();
+String msg = null;
+if (header != null) {
+if (body != null && !header.equals(body)) {
+msg = header + " (" + body + ")";
+} else {
+msg = header;
+}
+} else {
+msg = body;
+}
+
+return tr(""
++ "Authorisation at the OSM server failed."
++ "The server reported the following error:"
++ "''{0}''"
++ "",
+msg
+);
+}
+
 public static String explainFailedOAuthAuthorisation(OsmApiException e) {
 e.printStackTrace();
 return tr(""


signature.asc
Description: PGP signature