Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

Bug#972076: tasksel suggests the use of mktemp instead of tempfile

175 views
Skip to first unread message

Markus Steinko

unread,
Oct 12, 2020, 5:00:03 AM10/12/20
to
Package: tasksel
Version: 3.59
Severity: normal

Dear Maintainer,

*** Reporter, please consider answering these questions, where
appropriate ***

   * What led up to the situation?

        Using tasksel to install some task, or with the --new-install
option
        ends successfully, but with a warning:
        WARNING: tempfile is deprecated; consider using mktemp instead.

   * What exactly did you do (or not do) that was effective (or
     ineffective)?

        Just read through the postinst script to see how paths get handled.

   * What was the outcome of this action?

        I could not find what I was looking for, as tasksel maybe
depends on
        another package which is responsible for the warning.

   * What outcome did you expect instead?

*** End of the template - remove these template lines ***



-- System Information:
Debian Release: bullseye/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (500, 'testing')
Architecture: amd64 (x86_64)

Kernel: Linux 5.8.0-2-amd64 (SMP w/4 CPU threads)
Kernel taint flags: TAINT_PROPRIETARY_MODULE, TAINT_OOT_MODULE,
TAINT_UNSIGNED_MODULE
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8), LANGUAGE
not set
Shell: /bin/sh linked to /usr/bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages tasksel depends on:
ii  apt                     2.1.10
ii  debconf [debconf-2.0]   1.5.74
ii  liblocale-gettext-perl  1.07-4
ii  perl-base               5.30.3-4
ii  tasksel-data            3.59

tasksel recommends no packages.

tasksel suggests no packages.

-- debconf information:
  tasksel/tasks: desktop, gnome-desktop, cinnamon-desktop,
mate-desktop, ssh-server
  tasksel/desktop:
  tasksel/first: desktop, gnome-desktop, ssh-server, standard
  tasksel/title:

Sven Joachim

unread,
Oct 12, 2020, 1:30:03 PM10/12/20
to
Control: severity -1 minor
Control: tags -1 patch

Am 12.10.2020 um 10:45 schrieb Markus Steinko:

> Package: tasksel
> Version: 3.59
> Severity: normal
>
> Dear Maintainer,
>
> *** Reporter, please consider answering these questions, where
> appropriate ***
>
>    * What led up to the situation?
>
>         Using tasksel to install some task, or with the --new-install
> option
>         ends successfully, but with a warning:
>         WARNING: tempfile is deprecated; consider using mktemp instead.

The warning comes from tempfile itself, added in debianutils 4.10. The
obvious, but untested attached patch fixes that, although it might be
better to use File::Temp instead (in perl-base since version 5.20.1-3).

Cheers,
Sven

0001-Avoid-tempfile-deprecation-warning.patch

Holger Wansing

unread,
Oct 13, 2020, 3:30:04 AM10/13/20
to
Hi,

Sven Joachim <sven...@gmx.de> wrote:
> >         ends successfully, but with a warning:
> >         WARNING: tempfile is deprecated; consider using mktemp instead.
>
> The warning comes from tempfile itself, added in debianutils 4.10. The
> obvious, but untested attached patch fixes that, although it might be
> better to use File::Temp instead (in perl-base since version 5.20.1-3).

tasksel builds fine with Sven's "tempfile -> mktemp" patch applied, so any
objections against this?

Holger


--
Holger Wansing <hwan...@mailbox.org>
PGP-Fingerprint: 496A C6E8 1442 4B34 8508 3529 59F1 87CA 156E B076

Markus Steinko

unread,
Oct 15, 2020, 12:50:03 AM10/15/20
to
When rechecking for the warning I figured out that `tasksel
--new-install` also gives an error. I'm not sure if this is because of
already having all needed and available packages that option offers
installed, or if it maybe is another bug.


sudo tasksel --new-install
Invalid entry length (16). Fixed up to 11.
Invalid entry length (16). Fixed up to 11.
Invalid entry length (16). Fixed up to 11.
WARNING: tempfile is deprecated; consider using mktemp instead.
tasksel: Aptitude fehlgeschlagen (100)


Regards,

Holger Wansing

unread,
Nov 5, 2020, 2:20:04 PM11/5/20
to
Hi,

Holger Wansing <hwan...@mailbox.org> wrote:
>
> Sven Joachim <sven...@gmx.de> wrote:
> > >         ends successfully, but with a warning:
> > >         WARNING: tempfile is deprecated; consider using mktemp instead.
> >
> > The warning comes from tempfile itself, added in debianutils 4.10. The
> > obvious, but untested attached patch fixes that, although it might be
> > better to use File::Temp instead (in perl-base since version 5.20.1-3).
>
> tasksel builds fine with Sven's "tempfile -> mktemp" patch applied, so any
> objections against this?

Just pushed.

nicoo

unread,
Nov 17, 2020, 5:10:04 PM11/17/20
to
Control: fixed -1 3.60

Hi Sven,

On Thu, Nov 05, 2020 at 08:14:56PM +0100, Holger Wansing wrote:
> Holger Wansing <hwan...@mailbox.org> wrote:
> > tasksel builds fine with Sven's "tempfile -> mktemp" patch applied, so any
> > objections against this?
>
> Just pushed.

Could you confirm that the version Holdger uploaded fixes the issue?
If it doesn't, please remove the “fixed in v3.60” tag I just added.


Best,

nicoo
signature.asc

Holger Wansing

unread,
Nov 19, 2020, 4:30:04 AM11/19/20
to
Hi,

"Debian Bug Tracking System" <ow...@bugs.debian.org> wrote:
> On 2020-11-17 23:00 +0100, nicoo wrote:
>
> > Control: fixed -1 3.60
> >
> > Hi Sven,
> >
> > On Thu, Nov 05, 2020 at 08:14:56PM +0100, Holger Wansing wrote:
> >> Holger Wansing <hwan...@mailbox.org> wrote:
> >> > tasksel builds fine with Sven's "tempfile -> mktemp" patch applied, so any
> >> > objections against this?
> >>
> >> Just pushed.
> >
> > Could you confirm that the version Holdger uploaded fixes the issue?
>
> Sure it does. :-)
>
> > If it doesn't, please remove the “fixed in v3.60” tag I just added.
>
> On the contrary, I am closing the bug hereby, no idea why Holger did not
> do so with the upload.

Because there is - strictly spoken - one aspect of the bug, which is not yet
fixed. That is:


---snip-------

Date: Thu, 15 Oct 2020 06:43:05 +0200
From: Markus Steinko <ste...@gmail.com>
To: 972...@bugs.debian.org
Subject: Bug#972076: tasksel --new-install


When rechecking for the warning I figured out that `tasksel
--new-install` also gives an error. I'm not sure if this is because of
already having all needed and available packages that option offers
installed, or if it maybe is another bug.


sudo tasksel --new-install
Invalid entry length (16). Fixed up to 11.
Invalid entry length (16). Fixed up to 11.
Invalid entry length (16). Fixed up to 11.
WARNING: tempfile is deprecated; consider using mktemp instead.
tasksel: Aptitude fehlgeschlagen (100)

---snap-------


You can - of course - say, that is another issue, and thus #972076 can be closed,
but then the other issue should get a separate bugreport.

nicoo

unread,
Nov 19, 2020, 6:00:04 AM11/19/20
to
Control: clone -1 -2
Control: retitle -2 tasksel --new-install fails
Control: severity -2 normal

Hi,

On Thu, Nov 19, 2020 at 10:21:29AM +0100, Holger Wansing wrote:
> Swen Joachim <sven...@gmx.de> wrote:
> > On 2020-11-17 23:00 +0100, nicoo wrote:
> > > Could you confirm that the version Holdger uploaded fixes the issue?
> > Sure it does. :-)
> >
> > > If it doesn't, please remove the “fixed in v3.60” tag I just added.
> > On the contrary, I am closing the bug hereby, no idea why Holger did not
> > do so with the upload.
>
> Because there is - strictly spoken - one aspect of the bug, which is not yet
> fixed. That is:
>
> Date: Thu, 15 Oct 2020 06:43:05 +0200
> From: Markus Steinko <ste...@gmail.com>
> To: 972...@bugs.debian.org
> Subject: Bug#972076: tasksel --new-install
>
>
> When rechecking for the warning I figured out that `tasksel
> --new-install` also gives an error.

Oh, apologies for missing that part.

> I'm not sure if this is because of
> already having all needed and available packages that option offers
> installed, or if it maybe is another bug.

Sounds like another bug, but I'll try to reproduce with the new tasksel.

> You can - of course - say, that is another issue, and thus #972076 can be
> closed, but then the other issue should get a separate bugreport.

Agreed, cloning this report as I do not see a separate one in the BTS yet.


Best,

nicoo
signature.asc

Holger Wansing

unread,
Dec 6, 2020, 9:40:04 AM12/6/20
to
Control: reopen -1
Control: notfound -1 3.61
The bug has been cloned, however the new one is also closed already.
So reopening.

Additionally, I tested with tasksel 3.61 from testing, if this bug is still
reproducible, and found that it is not!
Everything worked fine there:
I called "tasksel --new-install", got the selection menu, chose to additionally
install ssh-server, and that worked fine.

So updating the list of not-affected versions.
Let's see what happens with future versions ...


(FTR: since the point of this bugreport may be overseen in the bug log:
please find the description of the problem in
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=975233#24 )

Holger Wansing

unread,
Dec 7, 2020, 4:00:05 PM12/7/20
to
Control: reopen -1
Control: notfound -1 3.61

[Re-sending this to the correct bug now]
The bug has been cloned, however the new one is also closed already.
So reopening.

Additionally, I tested with tasksel 3.61 from testing, if this bug is still
reproducible, and found that it is not!
Everything worked fine there:
I called "tasksel --new-install", got the selection menu, chose to additionally
install ssh-server, and that worked fine.

So updating the list of not-affected versions.
Let's see what happens with future versions ...


(FTR: since the point of this bugreport may be overseen in the bug log:
please find the description of the problem in
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=975233#24 )


Holger Wansing

unread,
Jul 17, 2022, 9:50:03 AM7/17/22
to
Hi,

Holger Wansing <hwan...@mailbox.org> wrote (Sun, 6 Dec 2020 15:29:05 +0100):
> Additionally, I tested with tasksel 3.61 from testing, if this bug is still
> reproducible, and found that it is not!
> Everything worked fine there:
> I called "tasksel --new-install", got the selection menu, chose to additionally
> install ssh-server, and that worked fine.
>
> So updating the list of not-affected versions.
> Let's see what happens with future versions ...

Tested today with tasksel 3.69+rebuild from unstable:
this issue is still unreproducible, so let's close this bug.
0 new messages