[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: New bali-phy version 3.6.0 and some lintian questions



Hi Benjamin,

thanks for the explanation.  I do not see any reason to deal explicitly
with the empty directory and thus I uploaded.

Thanks a lot for your work

    Andreas.

On Sat, Feb 06, 2021 at 07:12:26PM -0800, Benjamin Redelings wrote:
> Hi Andreas,
> 
> Thanks for the feedback!  I pushed my changes to salsa, and I think I have
> incorporated all of your feedback except about the empty directory.
> 
> > > I: bali-phy: package-contains-empty-directory
> > > usr/share/doc/bali-phy/examples/models/regresssion/
> > Well, is this really intended to have an empty directory here?  WHat
> > is the purpose?
> 
> So models/regression is a symlink, and it looks like meson doesn't install
> it.  This is an installation bug that I never noticed before.  Meson creates
> the directory that the symlink points to, but does not put anything inside
> it.  Meson will soon stop following symlinks during installation, so fixing
> this behavior in meson is probably not worth it.
> 
> In practice, the missing directory doesn't matter.  I am probably going to
> delete the symlink in the next major upstream version.  I don't want to make
> an upstream point release to fix this, since it is not really broken.
> 
> However, I looked at two things to make the warning message go away:
> 
> - I tried to remove the symlink with quilt, but quilt won't handle deleting
> a symlink
> 
> - The lintian docs suggest adding `find path/to/base/dir -type d -empty
> -delete` to debian/rules, but its not clear that there is always a
> hard-coded path/to/base/dir.
> 
> I could add a lintian override, also.  What do you prefer?
> 
> -BenRI
> 
> 
> 

-- 
http://fam-tille.de


Reply to: