Bug#699171: Pre-Approval: capi4hylafax/1:01.03.00.99.svn.300-19

2013-03-19 Thread Julien Cristau
On Mon, Mar 18, 2013 at 22:22:03 +0100, Joachim Wiedorn wrote:

 Hello Julien,
 
 Julien Cristau wrote on 2013-03-18 20:42:
 
  The debdiff and the above is rather short on explanations (and I'd
  rather not read the whole bug log for 661482)...  Care to explain why
  these directories must be created in postinst rather than shipped in the
  package?
 
 hylafax itself have the working directory /var/spool/hylafax with many
 subdirectories. To make capi4hylafax working together with hylafax it 
 must use these directories, too. But there is another case: capi4hylafax
 can also work without hylafax.
 
 The main problem is: while deinstalling hylafax the package hylafax
 doesn't know something about capi4hylafax. If the package capi4hylafax
 would shipping these named directories of hylafax in its package and
 hylafax would be purged then this give errors because the directory
 /var/spool/hylafax with some subdirectories would be owned by both
 packages and hylafax want to delete these directories.
 
I don't understand.  dpkg won't try to remove a directory owned by two
packages if you remove one of them.

Cheers,
Julien


signature.asc
Description: Digital signature


Bug#699171: Pre-Approval: capi4hylafax/1:01.03.00.99.svn.300-19

2013-03-19 Thread Joachim Wiedorn
Hello Julien,

Julien Cristau wrote on 2013-03-19 10:36:

 I don't understand.  dpkg won't try to remove a directory owned by two
 packages if you remove one of them.

It isn't done by dpkg but by postrm of hylafax-server package in this way:

[ -d /var/spool/hylafax/etc ]  rm -rf /var/spool/hylafax/etc
[ -L /var/spool/hylafax/bin ]  rm /var/spool/hylafax/bin

for i in /etc/hylafax/setup.cache /etc/hylafax/setup.modem \
/var/spool/hylafax/status/any.info /var/spool/hylafax/dev/null \
/var/spool/hylafax/FIFO /var/spool/hylafax/bin/ps2fax \
/var/spool/hylafax/bin/pdf2fax /var/spool/hylafax/bin/bin \
/etc/default/hylafax
do
[ -e $i -o -L $i ]  rm $i
done

[ -d /var/spool/hylafax/bin ]  rmdir --ignore-fail-on-non-empty 
/var/spool/hylafax/bin

With hylafax 6.0.6-5 the postrm script do not remove /var/spool/hylafax anymore.
So capi4hylafax have the chance to use this directory without problems.


---
Have a nice day.

Joachim (Germany)


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



Bug#699171: Pre-Approval: capi4hylafax/1:01.03.00.99.svn.300-19

2013-03-18 Thread Julien Cristau
On Mon, Jan 28, 2013 at 15:11:42 +0100, Joachim Wiedorn wrote:

 Package: release.debian.org
 Severity: normal
 User: release.debian@packages.debian.org
 Usertags: pre-approval
 
 Hello release team,
 
 I ask for pre-approval for package capi4hylafax.
 
 There is a difficult RC bug (#661482) which mention also package hylafax.
 To resolve this RC bug a bugfix for hylafax and capi4hylafax is needed. 
 
 As first step here is the bugfix of capi4hylafax:
 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697733
 
 The updated capi4hylafax 1:01.03.00.99.svn.300-19 fixes a part of RC bug
 #661482:
 
   * Optimize postrm script and add new postinst script to work together
   with hylafax-server and its 'mountbind'. (See: #661482)
 
 Details are in the attached debdiff.
 
The debdiff and the above is rather short on explanations (and I'd
rather not read the whole bug log for 661482)...  Care to explain why
these directories must be created in postinst rather than shipped in the
package?

Cheers,
Julien


signature.asc
Description: Digital signature


Bug#699171: Pre-Approval: capi4hylafax/1:01.03.00.99.svn.300-19

2013-03-18 Thread Joachim Wiedorn
Hello Julien,

Julien Cristau wrote on 2013-03-18 20:42:

 The debdiff and the above is rather short on explanations (and I'd
 rather not read the whole bug log for 661482)...  Care to explain why
 these directories must be created in postinst rather than shipped in the
 package?

hylafax itself have the working directory /var/spool/hylafax with many
subdirectories. To make capi4hylafax working together with hylafax it 
must use these directories, too. But there is another case: capi4hylafax
can also work without hylafax.

The main problem is: while deinstalling hylafax the package hylafax
doesn't know something about capi4hylafax. If the package capi4hylafax
would shipping these named directories of hylafax in its package and
hylafax would be purged then this give errors because the directory
/var/spool/hylafax with some subdirectories would be owned by both
packages and hylafax want to delete these directories.

To manage all the different cases the only way is the handling in the
pre* post* scripts of capi4hylafax.

I hope these information helps.
I think Giuseppe could explain some more details in better english.

---
Have a nice day.

Joachim (Germany)


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



Bug#699171: Pre-Approval: capi4hylafax/1:01.03.00.99.svn.300-19

2013-01-28 Thread Joachim Wiedorn
Package: release.debian.org
Severity: normal
User: release.debian@packages.debian.org
Usertags: pre-approval

Hello release team,

I ask for pre-approval for package capi4hylafax.

There is a difficult RC bug (#661482) which mention also package hylafax.
To resolve this RC bug a bugfix for hylafax and capi4hylafax is needed. 

As first step here is the bugfix of capi4hylafax:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=697733

The updated capi4hylafax 1:01.03.00.99.svn.300-19 fixes a part of RC bug
#661482:

  * Optimize postrm script and add new postinst script to work together
  with hylafax-server and its 'mountbind'. (See: #661482)

Details are in the attached debdiff.

Many thanks for your work,

---
Have a nice day.

Joachim (Germany)

debdiff_capi4hylafax_300-19.diff.gz
Description: GNU Zip compressed data