Bug#985292: materia-gtk-theme: unhandled symlink to directory conversion: /usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets

2021-04-11 Thread Leandro Cunha
Hi,

On Sat, Apr 3, 2021 at 5:14 PM Andreas Beckmann  wrote:

> On 03/04/2021 07.43, Leandro Cunha wrote:
> > Can you test the version I pushed for Salsa and confirm that the problem
> > has been fixed?
> >
> > [1] https://salsa.debian.org/leandrocunha/materia-gtk-theme
>
> There is no releated change in git, only a changelog entry (I would have
> expected a .maintscript file to be added).
> But there is a revert of debian/rules to an older version (and from
> short debhelper 13 to something much older), which is probably unwanted
> and inappropriate at this point of the release cycle.
>
>
> Andreas
>

I made an update to the repository in Salsa based on the log made available.
Please confirm the correction with a log and this test that was done didn't
find
documentation of it to reproduce here. Thanks.

[1] https://salsa.debian.org/leandrocunha/materia-gtk-theme

-- 
Cheers,
Leandro Cunha
Debian Contributor and developer.


Bug#985292: materia-gtk-theme: unhandled symlink to directory conversion: /usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets

2021-04-03 Thread Andreas Beckmann

On 03/04/2021 07.43, Leandro Cunha wrote:

Can you test the version I pushed for Salsa and confirm that the problem
has been fixed?

[1] https://salsa.debian.org/leandrocunha/materia-gtk-theme


There is no releated change in git, only a changelog entry (I would have 
expected a .maintscript file to be added).
But there is a revert of debian/rules to an older version (and from 
short debhelper 13 to something much older), which is probably unwanted 
and inappropriate at this point of the release cycle.



Andreas



Bug#985292: materia-gtk-theme: unhandled symlink to directory conversion: /usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets

2021-04-02 Thread Leandro Cunha
Hi,

Can you test the version I pushed for Salsa and confirm that the problem
has been fixed?

[1] https://salsa.debian.org/leandrocunha/materia-gtk-theme
[2] https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=985292

-- 
Cheers,
Leandro Cunha
Debian Contributor and developer.


Bug#985292: materia-gtk-theme: unhandled symlink to directory conversion: /usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets

2021-03-15 Thread Andreas Beckmann
Package: materia-gtk-theme
Version: 20200916-0.1
Severity: serious
User: debian...@lists.debian.org
Usertags: piuparts

Hi,

an upgrade test with piuparts revealed that your package installs files
over existing symlinks and possibly overwrites files owned by other
packages. This usually means an old version of the package shipped a
symlink but that was later replaced by a real (and non-empty)
directory. This kind of overwriting another package's files cannot be
detected by dpkg.

This was observed on the following upgrade paths:

  buster -> bullseye

For /usr/share/doc/PACKAGE this may not be problematic as long as both
packages are installed, ship byte-for-byte identical files and are
upgraded in lockstep. But once one of the involved packages gets
removed, the other one will lose its documentation files, too,
including the copyright file, which is a violation of Policy 12.5:
https://www.debian.org/doc/debian-policy/ch-docs.html#copyright-information

For other overwritten locations anything interesting may happen.

Note that dpkg intentionally does not replace directories with symlinks
and vice versa, you need the maintainer scripts to do this.
See in particular the end of point 4 in
https://www.debian.org/doc/debian-policy/ch-maintainerscripts.html#details-of-unpack-phase-of-installation-or-upgrade

It is recommended to use the dpkg-maintscript-helper commands
'dir_to_symlink' and 'symlink_to_dir' (available since dpkg 1.17.14)
to perform the conversion, ideally using d/$PACKAGE.maintscript.
See dpkg-maintscript-helper(1) and dh_installdeb(1) for details.


>From the attached log (scroll to the bottom...):

1m8.4s ERROR: installs objects over existing directory symlinks:
  
/usr/share/themes/Materia-compact/gtk-3.0/assets/scale-horz-marks-after-slider-dark.png
 (materia-gtk-theme) != 
/usr/share/themes/Materia-compact/gtk-assets/scale-horz-marks-after-slider-dark.png
 (?)
/usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets
  
/usr/share/themes/Materia-compact/gtk-3.0/assets/scale-horz-marks-after-slider-d...@2.png
 (materia-gtk-theme) != 
/usr/share/themes/Materia-compact/gtk-assets/scale-horz-marks-after-slider-d...@2.png
 (?)
/usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets
  
/usr/share/themes/Materia-compact/gtk-3.0/assets/scale-horz-marks-after-slider-disabled-dark.png
 (materia-gtk-theme) != 
/usr/share/themes/Materia-compact/gtk-assets/scale-horz-marks-after-slider-disabled-dark.png
 (?)
/usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets
  
/usr/share/themes/Materia-compact/gtk-3.0/assets/scale-horz-marks-after-slider-disabled-d...@2.png
 (materia-gtk-theme) != 
/usr/share/themes/Materia-compact/gtk-assets/scale-horz-marks-after-slider-disabled-d...@2.png
 (?)
/usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets
  
/usr/share/themes/Materia-compact/gtk-3.0/assets/scale-horz-marks-after-slider-disabled.png
 (materia-gtk-theme) != 
/usr/share/themes/Materia-compact/gtk-assets/scale-horz-marks-after-slider-disabled.png
 (?)
/usr/share/themes/Materia-compact/gtk-3.0/assets -> ../gtk-assets
[...]
  
/usr/share/themes/Materia/gtk-3.0/assets/selectionmode-checkbox-unchecked-d...@2.png
 (materia-gtk-theme) != 
/usr/share/themes/Materia/gtk-assets/selectionmode-checkbox-unchecked-d...@2.png
 (?)
/usr/share/themes/Materia/gtk-3.0/assets -> ../gtk-assets
  /usr/share/themes/Materia/gtk-3.0/assets/selectionmode-checkbox-unchecked.png 
(materia-gtk-theme) != 
/usr/share/themes/Materia/gtk-assets/selectionmode-checkbox-unchecked.png (?)
/usr/share/themes/Materia/gtk-3.0/assets -> ../gtk-assets
  
/usr/share/themes/Materia/gtk-3.0/assets/selectionmode-checkbox-unchec...@2.png 
(materia-gtk-theme) != 
/usr/share/themes/Materia/gtk-assets/selectionmode-checkbox-unchec...@2.png (?)
/usr/share/themes/Materia/gtk-3.0/assets -> ../gtk-assets


cheers,

Andreas


materia-gtk-theme_20200916-0.1.log.gz
Description: application/gzip