Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.debian.maint.python > #15072

Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)"

Show key headers only | View raw

On Fri, 28 Jul 2023 at 11:53:29 +0200, Carsten Schoenert wrote:
> To quote from the BTS:
> ---%<---
> > In 1.2 upstream added a test suite. We should run it during build
> > (cd common && $(MAKE) test) but it needs to be able to write to the home
> > directory, which is disabled on Debian auto-builders. Need to find
> > a solution to that.
> --->%---
> 
> To me it's clear what the problem is. The test requires a $HOME folder, but
> the build environment doesn't provide something like this.

If backintime uses $HOME, and doesn't rely on $HOME being the same as
$(getent passwd $(id -u)|cut -d: -f6), then it might actually be possible
to run its test suite with a dependency on a suitably new debhelper.

In debhelper compatibility level 13, dh_auto_test sets $HOME to a temporary
directory (#942111) which might well be enough to run the test suite
non-destructively. If that's sufficient, I'm sure the maintainer of the
backintime package would appreciate a tested patch sent to #940319.
The way to test this would be to build backintime in sbuild, with a uid
whose "official" home directory in /etc/passwd doesn't exist in the chroot.

The other angle this could be approached from is as an upstream developer:
as an upstream, would you really want running the backintime test suite
to make potentially destructive changes to your real home directory? As
an upstream developer of other packages, I wouldn't want that: if I have
made an implementation mistake, I want to be able to find out about that
by running the test suite, knowing that the test suite won't damage my
real home directory.

Making the test suite write to a mock home directory instead of to the
real home directory, and changing or unsetting environment variables that
point to the real home directory (again, see #942111) during automated
testing, would make the test suite safer and more predictable.

    smcv

Back to linux.debian.maint.python | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[backintime] Switch the maintainer to "Debian Python Team (DPT)" c.buhtz@posteo.jp - 2023-06-12 10:00 +0200
  Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" Jonathan Carter <jcc@debian.org> - 2023-06-12 10:50 +0200
    Re: [backintime] Switch the maintainer to "Debian Python Team  (DPT)" <c.buhtz@posteo.jp> - 2023-07-23 09:10 +0200
      Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" Carsten Schoenert <c.schoenert@t-online.de> - 2023-07-23 09:50 +0200
        Re: [backintime] Switch the maintainer to "Debian Python Team  (DPT)" <c.buhtz@posteo.jp> - 2023-07-27 22:00 +0200
          Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" c.buhtz@posteo.jp - 2023-07-28 09:20 +0200
          Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" Carsten Schoenert <c.schoenert@t-online.de> - 2023-07-28 12:00 +0200
            Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" c.buhtz@posteo.jp - 2023-07-28 13:10 +0200
              Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" Simon McVittie <smcv@debian.org> - 2023-07-28 14:40 +0200
              Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" Andrey Rakhmatullin <wrar@wrar.name> - 2023-07-28 15:40 +0200
            Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" Simon McVittie <smcv@debian.org> - 2023-07-28 14:50 +0200
              Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" c.buhtz@posteo.jp - 2023-07-28 15:00 +0200
            Re: [backintime] Switch the maintainer to "Debian Python Team (DPT)" Nicholas D Steeves <sten@debian.org> - 2023-07-28 21:40 +0200

csiph-web