Your message dated Sat, 11 Jan 2025 11:03:09 +0000
with message-id <e1twzgn-009jzw...@coccia.debian.org>
and subject line Close 1068762
has caused the Debian Bug report #1068762,
regarding bookworm-pu: package oar/2.5.9-1+deb12u1
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


-- 
1068762: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=1068762
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
Tags: bookworm
X-Debbugs-Cc: o...@packages.debian.org, pierre.ney...@free.fr
Control: affects -1 + src:oar
User: release.debian....@packages.debian.org
Usertags: pu

[ Reason ]
The oar package is suffering several issues in bookworm.
The root cause is that we forgot to update it in time with the release 
schedule, so the version in bookworm is the one in bullseye.

The testing version is a new upstream version (that fixes the following 
problems and also has other improvments)

We would like you to allow us to fix the following problems in stable:
- one of the package is a php program that does not work with php8 (bug 
#1068444)
- adduser >= 3.130 (ie the one in bookworm) changes the way adduser's 
--disabled-(login|password) options work.
  So, a fix when creating users is required (note: this bug appears only on 
fresh oar install, not on upgrade) (bug #1068713)
- a binary package (oar-web-status) is missing a dependency on libcgi-fast-perl 
(bug #1068711)

Note that #1068444 and #1068713 are fixed in testing for a long time (since the 
upload of 2.5.10-1)
#1068711 has just been noticed, with a upload in unstable a few minutes ago (in 
2.5.10-2)

[ Impact ]
Without this upload:
- the draw-gantt webapp does not work at all in bookworm (php8 incompatibility)
- new installation of oar system cannot submit jobs (until the password field 
in /etc/shadow is manually fixed)
- libcgi-fast-perl must be manually installed if oar-web-status is to be used
  (this last bug is often hidden: if oar-restful-api is also installed, then 
the libcgi-fast-perl is installed)

[ Tests ]
Manual tests has been conducted upstream (and fixes have been in upstream 
2.5.10 for a long time)

[ Risks ]
Modifications seems trivial and, except for the dependencies, used on 
production systems for several months.

[ Checklist ]
  [X] *all* changes are documented in the d/changelog
  [X] I reviewed all changes and I approve them
  [X] attach debdiff against the package in (old)stable
  [X] the issue is verified as fixed in unstable

[ Changes ]
See previous explainations and debdiff.

  Regards
    Vincent
diff -Nru oar-2.5.9/debian/changelog oar-2.5.9/debian/changelog
--- oar-2.5.9/debian/changelog  2021-01-11 16:44:43.000000000 +0100
+++ oar-2.5.9/debian/changelog  2024-04-10 14:02:42.000000000 +0200
@@ -1,3 +1,16 @@
+oar (2.5.9-1+deb12u1) bookworm; urgency=medium
+
+  [ Pierre Neyron ]
+  * Fix Drawgantt-SVG with php8 (Closes: #1068444)
+    - create_function does not exists anymore
+    - static methods must be declared as such
+  * Fix the oar user creation on new install (locked otherwise) (Closes:
+    #1068713)
+  * oar-web-status: add missing dependency to libcgi-fast-perl (Closes:
+    #1068711)
+
+ -- Vincent Danjean <vdanj...@debian.org>  Wed, 10 Apr 2024 14:02:42 +0200
+
 oar (2.5.9-1) unstable; urgency=medium
 
   * New release
diff -Nru oar-2.5.9/debian/patches/drawgantt-svg_fixes_for_php_8.patch 
oar-2.5.9/debian/patches/drawgantt-svg_fixes_for_php_8.patch
--- oar-2.5.9/debian/patches/drawgantt-svg_fixes_for_php_8.patch        
1970-01-01 01:00:00.000000000 +0100
+++ oar-2.5.9/debian/patches/drawgantt-svg_fixes_for_php_8.patch        
2024-04-10 14:02:42.000000000 +0200
@@ -0,0 +1,45 @@
+[drawgantt-svg] Fixes for PHP 8
+
+Backported from upstream 2.5.10
+
+Fix fatal errors occuring with PHP8:
+- The create_function function is REMOVED as of PHP 8.0.0. 
(https://www.php.net/manual/en/function.create-function.php)
+- Non-static methods Shuffle::Init() and Shuffle::get_int() cannot be called 
statically. Need to make them static.
+
+From: Pierre Neyron <pierre.ney...@free.fr>
+
+diff --git 
a/sources/visualization_interfaces/DrawGantt-SVG/drawgantt-svg.php.in 
b/sources/visualization_interfaces/DrawGantt-SVG/drawgantt-svg.php.in
+index 5fd3b636..e24d3faa 100644
+--- a/sources/visualization_interfaces/DrawGantt-SVG/drawgantt-svg.php.in
++++ b/sources/visualization_interfaces/DrawGantt-SVG/drawgantt-svg.php.in
+@@ -269,10 +269,10 @@ function date2px($date) {
+ // class to compute colors for jobs
+ class Shuffle {
+     protected static $singleton;
+-    function init($shuffle_instance) {
++    static function init($shuffle_instance) {
+         self::$singleton = $shuffle_instance;
+     }
+-    function get_int($job) {
++    static function get_int($job) {
+         if (self::$singleton === null) {
+             self::init(new Shuffle());
+         }
+@@ -603,7 +603,7 @@ class Resource {
+       if ($this->type == $CONF['resource_group_level']) {
+         $first_resource_base = true;
+       }
+-      usort($this->children, create_function('$a,$b','return $b->cmp($a);'));
++      usort($this->children, function($a,$b) { return $b->cmp($a); });
+       foreach ($this->children as $child) {
+         $h += $child->svg_hierarchy($x+$CONF['hierarchy_resource_width'], 
$y+$h, $labels);
+       }
+@@ -650,7 +650,7 @@ class Resource {
+       }
+     }
+     foreach ($states as $value => $date_weight_array) {
+-      usort($date_weight_array, create_function('$a,$b','list($d1,$w1)=$a; 
list($d2,$w2)=$b; return ($d1 != $d2)?(($d1 < $d2)?-1:1):(($w1 != $w2)?(($w1 < 
$w2)?1:-1):0);'));
++      usort($date_weight_array, function($a,$b) { list($d1,$w1)=$a; 
list($d2,$w2)=$b; return ($d1 != $d2)?(($d1 < $d2)?-1:1):(($w1 != $w2)?(($w1 < 
$w2)?1:-1):0); });
+       $weight = 0;
+       foreach ($date_weight_array as $dwr) {
+         list ($d,$w,$r) = $dwr;
diff -Nru 
oar-2.5.9/debian/patches/fix_the_oar_user_creation_with_debian12_adduser_3.130.patch
 
oar-2.5.9/debian/patches/fix_the_oar_user_creation_with_debian12_adduser_3.130.patch
--- 
oar-2.5.9/debian/patches/fix_the_oar_user_creation_with_debian12_adduser_3.130.patch
        1970-01-01 01:00:00.000000000 +0100
+++ 
oar-2.5.9/debian/patches/fix_the_oar_user_creation_with_debian12_adduser_3.130.patch
        2024-04-10 14:02:42.000000000 +0200
@@ -0,0 +1,24 @@
+[setup] Fix the oar user creation with Debian12 (adduser > 3.130)
+
+Backported from upstream 2.5.10
+
+adduser >= 3.130 changes to way adduser's --disabled-(login|password) options 
work.
+Need to force the "disabled password flag to '*' instead of '!', so that ssh 
connection are allowed.
+
+From: Pierre Neyron <pierre.ney...@free.fr>
+
+diff --git a/setup/common.sh.in b/setup/common.sh.in
+index a2eef6f7..bbfdf74e 100644
+--- a/setup/common.sh.in
++++ b/setup/common.sh.in
+@@ -30,6 +30,10 @@ create_oar_user() {
+                         --no-create-home \
+                         --disabled-password \
+                         ${OAROWNER} 2>/dev/null || true
++                # Force the "disabled password flag to '*' instead of '!'.
++                # Required in Debian12 (adduser >= 3.130), which changes to 
way
++                # adduser's --disabled-(login|password) options work.
++                usermod -p '*' ${OAROWNER} || true
+                 ;;
+             *)
+                 useradd --system \
diff -Nru oar-2.5.9/debian/patches/series oar-2.5.9/debian/patches/series
--- oar-2.5.9/debian/patches/series     2021-01-11 16:44:43.000000000 +0100
+++ oar-2.5.9/debian/patches/series     2024-04-10 14:02:42.000000000 +0200
@@ -0,0 +1,2 @@
+fix_the_oar_user_creation_with_debian12_adduser_3.130.patch
+drawgantt-svg_fixes_for_php_8.patch

--- End Message ---
--- Begin Message ---
Version: 12.9
This update has been released as part of 12.9. Thank you for your contribution.

--- End Message ---

Reply via email to