Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-10-06 Thread Ansgar Burchardt
Adam D. Barratt a...@adam-barratt.org.uk writes:
 On Sun, 2013-10-06 at 00:22 +0200, Holger Levsen wrote:
 hm, when running dak ls debian-edu-artwork on coccia.d.o I see the today 
 uploaded d-e-artwork package in the accepted suite, but this aint the case 
 for dak ls gosa - any idea why?
 
 Same for smbldap-tools / 0.9.7-1+deb7u1 / #718780 and slbackup-php / 
 0.4.3-2+deb7u1 / #711717 - both packages were also flagged for acceptance 
 but 
 are not in the accepted suite...

 accepted is basically an implementation detail of dak, for (aiui)
 packages that have been added to the archive since the previous
 dinstall.

The accepted suite contains the packages that show up on
http://incoming.debian.org, that is all uploads for ~36 hours after they
entered the archive.

If there were uploads to both p-u and unstable, we might remove the
lower version. This is probably a (minor) bug: dak dominate should just
ignore accepted.

So usually you can just ignore the accepted suite. It doesn't mean
anything interesting.

Ansgar


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-10-05 Thread Holger Levsen
Hi Adam,

On Samstag, 13. Juli 2013, Adam D. Barratt wrote:
   gosa (2.7.4-4.3~deb7u1) stable-updates; urgency=low
 fwiw, stable-updates is actually a different suite. :-) dak's mapping
 rules DTRT though, so flagged for acceptance.

hm, when running dak ls debian-edu-artwork on coccia.d.o I see the today 
uploaded d-e-artwork package in the accepted suite, but this aint the case 
for dak ls gosa - any idea why?

Same for smbldap-tools / 0.9.7-1+deb7u1 / #718780 and slbackup-php / 
0.4.3-2+deb7u1 / #711717 - both packages were also flagged for acceptance but 
are not in the accepted suite...
 

cheers,
Holger




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


Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-10-05 Thread Adam D. Barratt
On Sun, 2013-10-06 at 00:22 +0200, Holger Levsen wrote:
 hm, when running dak ls debian-edu-artwork on coccia.d.o I see the today 
 uploaded d-e-artwork package in the accepted suite, but this aint the case 
 for dak ls gosa - any idea why?
 
 Same for smbldap-tools / 0.9.7-1+deb7u1 / #718780 and slbackup-php / 
 0.4.3-2+deb7u1 / #711717 - both packages were also flagged for acceptance but 
 are not in the accepted suite...

accepted is basically an implementation detail of dak, for (aiui)
packages that have been added to the archive since the previous
dinstall.

The important thing is that the package is in proposed-updates, which
is what we monitor and what will become stable at the point release.

As an additional example:

$ dak ls sitesummary
sitesummary |0.0.68 |oldstable | source, all
sitesummary | 0.1.5 |   stable | source, all
sitesummary | 0.1.8 |  testing | source, all
sitesummary |  0.1.8+deb7u1 | proposed-updates | source, all
sitesummary | 0.1.9 | accepted | source, all
sitesummary | 0.1.9 |  buildd-unstable | source, all
sitesummary | 0.1.9 | unstable | source, all

i.e. the package in accepted currently is the more recently uploaded
sid package.

Regards,

Adam


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-07-13 Thread Holger Levsen
Hi Adam,

On Donnerstag, 11. Juli 2013, Adam D. Barratt wrote:
 Okay, thanks; please go ahead.

thanks, uploaded!

Changes: 
 gosa (2.7.4-4.3~deb7u1) stable-updates; urgency=low
 .
   * Upload to stable updates.
 .
 gosa (2.7.4-4.3) unstable; urgency=low
 .
   * Non-maintainer upload.
   * debian/patches/fix-mass-ldapimport.patch: New patch, fixes LDAP mass
 import, by Giorgio Pioda and Petter Reinholdtsen. (Closes: #698840)


cheers,
Holger


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


Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-07-13 Thread Adam D. Barratt
On Sat, 2013-07-13 at 13:06 +0200, Holger Levsen wrote:
 On Donnerstag, 11. Juli 2013, Adam D. Barratt wrote:
  Okay, thanks; please go ahead.
 
 thanks, uploaded!
 
 Changes: 
  gosa (2.7.4-4.3~deb7u1) stable-updates; urgency=low

fwiw, stable-updates is actually a different suite. :-) dak's mapping
rules DTRT though, so flagged for acceptance.

Regards,

Adam


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-07-11 Thread Holger Levsen
Hi Adam,

you've only asked about this

On Mittwoch, 10. Juli 2013, Adam D. Barratt wrote:
 Out of interest, do we know how long ago the functionality was
 temporarily disabled?

which Pere answered. 

gosa 2.7.4-4.3 (the version from this very bugreport) is now available in 
jessie and has also been successfully tested in Debian Edu's wheezy-test 
suite as well as our alpha3 pre-release, so can we upload this as 
2.7.4-4.2+deb7u1 to stable-proposed-updates?


cheers,
Holger


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


Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-07-11 Thread Adam D. Barratt
Control: tags -1 + confirmed

On Wed, 2013-07-10 at 23:37 +0200, Petter Reinholdtsen wrote:
 [Adam D. Barratt]
  Out of interest, do we know how long ago the functionality was
  temporarily disabled?
 
 I do not know exactly when it was disabled, but I know it was working
 in Debian Edu Squeeze.  Some time after Squeeze froze gosa saw a major
 revision, and in that process the bulk import feature was broken.  So
 for us it is a regression, in addition to making Debian Edu very hard
 to deploy when each user in a school need to be created individually
 instead of in bulk.

Okay, thanks; please go ahead.

Regards,

Adam


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-07-10 Thread Adam D. Barratt
On Sat, 2013-06-22 at 09:05 +0200, Petter Reinholdtsen wrote:
 Here is a request to approve an upload of gosa to wheezy, to fix a bug
 Debian Edu need to have fixed in our Debian Edu Wheezy release.  The
[...]
 +--- gosa/ldapmanager/addons/ldapmanager/contentcsv.tpl   (revisjon 21231)
  gosa/ldapmanager/addons/ldapmanager/contentcsv.tpl   (arbeidskopi)
 +@@ -1,5 +1,3 @@
 +-font color='red'Temporary disabled/font
 +-!--
 + p
 +   {t}The CSV import plug-in provides methods to generate user accounts from 
 a file containing Comma Separated Values. The administrator can decide which 
 columns should be transfered to which attribute. Note that you must have at 
 least the UID, GIVENNAME and SURNAME set.{/t}

Out of interest, do we know how long ago the functionality was
temporarily disabled?

Regards,

Adam


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-07-10 Thread Petter Reinholdtsen
[Adam D. Barratt]
 Out of interest, do we know how long ago the functionality was
 temporarily disabled?

I do not know exactly when it was disabled, but I know it was working
in Debian Edu Squeeze.  Some time after Squeeze froze gosa saw a major
revision, and in that process the bulk import feature was broken.  So
for us it is a regression, in addition to making Debian Edu very hard
to deploy when each user in a school need to be created individually
instead of in bulk.

-- 
Happy hacking
Petter Reinholdtsen


-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-07-05 Thread Holger Levsen
Hi,

On Samstag, 22. Juni 2013, Petter Reinholdtsen wrote:
 Here is a request to approve an upload of gosa to wheezy, to fix a bug
 Debian Edu need to have fixed in our Debian Edu Wheezy release.  The
 same fix was uploaded as versoin 2.7.4-4.3 to unstable with this
 changelog entry:
 
* debian/patches/fix-mass-ldapimport.patch: New patch, fixes LDAP mass
  import, by Giorgio Pioda and Petter Reinholdtsen. (Closes: #698840)
 
 We need it to make sure schools can create users for all teachers and
 users in a school starting to use Debian Edu (or migrating to the new
 version) without having to create them one by one.

ping. Can we upload to s-p-u? The fix is in Debian Edu wheezy-test now too.


cheers,
Holger


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


Bug#713049: pu: gosa - fix mass import of users in wheezy

2013-06-22 Thread Petter Reinholdtsen

Package:  release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: unblock
User: debian-...@lists.debian.org
Usertags: debian-edu

Here is a request to approve an upload of gosa to wheezy, to fix a bug
Debian Edu need to have fixed in our Debian Edu Wheezy release.  The
same fix was uploaded as versoin 2.7.4-4.3 to unstable with this
changelog entry:

   * debian/patches/fix-mass-ldapimport.patch: New patch, fixes LDAP mass
 import, by Giorgio Pioda and Petter Reinholdtsen. (Closes: #698840)

We need it to make sure schools can create users for all teachers and
users in a school starting to use Debian Edu (or migrating to the new
version) without having to create them one by one.

Attached is the complete diff between version 2.7.4-4.3 in unstable and
version 2.7.4-4.2 in wheezy.

Please let us know if this is OK before we upload, and tell us which
version number to use for this upload.

-- 
Happy hacking
Petter Reinholdtsen
diff -Nru gosa-2.7.4/debian/changelog gosa-2.7.4/debian/changelog
--- gosa-2.7.4/debian/changelog	2013-02-07 21:28:31.0 +0100
+++ gosa-2.7.4/debian/changelog	2013-06-14 11:00:02.0 +0200
@@ -1,3 +1,11 @@
+gosa (2.7.4-4.3) unstable; urgency=low
+
+  * Non-maintainer upload.
+  * debian/patches/fix-mass-ldapimport.patch: New patch, fixes LDAP mass
+import, by Giorgio Pioda and Petter Reinholdtsen. (Closes: #698840)
+
+ -- Michael Banck mba...@debian.org  Fri, 14 Jun 2013 10:59:37 +0200
+
 gosa (2.7.4-4.2) unstable; urgency=low
 
   [ Jonathan Wiltshire ]
diff -Nru gosa-2.7.4/debian/patches/fix-mass-ldapimport.patch gosa-2.7.4/debian/patches/fix-mass-ldapimport.patch
--- gosa-2.7.4/debian/patches/fix-mass-ldapimport.patch	1970-01-01 01:00:00.0 +0100
+++ gosa-2.7.4/debian/patches/fix-mass-ldapimport.patch	2013-06-14 10:58:23.0 +0200
@@ -0,0 +1,48 @@
+Index: addons/ldapmanager/class_csvimport.inc
+===
+--- gosa/ldapmanager/addons/ldapmanager/class_csvimport.inc  (revisjon 21231)
 gosa/ldapmanager/addons/ldapmanager/class_csvimport.inc  (arbeidskopi)
+@@ -175,17 +175,25 @@
+ 
+ 
+ /* Set all submitted vars  */
+-if(is_array($keys))
++if(is_array($keys)){
+   foreach($keys as $key) {
+ if($key != status){
+   $this-usertab-$key = $dat[$key];
++  foreach($this-usertab-by_object as $pname = $plugin){
++if(isset($this-usertab-by_object[$pname]-$key)){
++  $this-usertab-by_object[$pname]-$key = $dat[$key];
++}
++  }
+ }
+   }
++}
+ 
+ /* Initialize template */
+ if($this-csvinfo['tplid']!=0){
+-  $this-usertab-adapt_from_template($this-csvinfo['arrtempls'][$this-csvinfo['tplid']]['dn']);
+-  $this-usertab-save_object();
++  $tpl_dn = $this-csvinfo['arrtempls'][$this-csvinfo['tplid']]['dn'];
++  $this-usertab-adapt_from_template($tpl_dn,array(uid,cn,givenName,sn));
++  $tpl_base  = preg_replace(/^[^,]+,.preg_quote(get_people_ou(), '/i')./, '', $tpl_dn);
++  $this-usertab-by_object['user']-base= $tpl_base;
+ } else {
+ 
+   /* Reset vars */
+Index: addons/ldapmanager/contentcsv.tpl
+===
+--- gosa/ldapmanager/addons/ldapmanager/contentcsv.tpl   (revisjon 21231)
 gosa/ldapmanager/addons/ldapmanager/contentcsv.tpl   (arbeidskopi)
+@@ -1,5 +1,3 @@
+-font color='red'Temporary disabled/font
+-!--
+ p
+   {t}The CSV import plug-in provides methods to generate user accounts from a file containing Comma Separated Values. The administrator can decide which columns should be transfered to which attribute. Note that you must have at least the UID, GIVENNAME and SURNAME set.{/t}
+ /p
+@@ -122,4 +120,3 @@
+   {/if}
+ /div
+ input type=hidden name=ignore
+---
diff -Nru gosa-2.7.4/debian/patches/series gosa-2.7.4/debian/patches/series
--- gosa-2.7.4/debian/patches/series	2012-06-19 09:57:50.0 +0200
+++ gosa-2.7.4/debian/patches/series	2013-06-08 18:58:37.0 +0200
@@ -5,3 +5,4 @@
 03_fix_class_mapping.patch
 04_fix_locale_location.patch
 no-image-warning.patch
+fix-mass-ldapimport.patch