Ticket #84 (closed Defect: fixed)

Opened 5 months ago

Last modified 2 months ago

Autopackage shouldn't assume user runs bash.

Reported by: Karatorian Assigned to: Jan-Nik
Priority: Low Milestone: 1.3
Component: main Version: 1.2.4
Severity: Trivial Keywords:
Cc:

Description

When installing the support package, the installer tells the user to run exec bash, even if their preferred shell is something other than bash.

Change History

05/29/08 03:14:45 changed by isak

  • priority changed from Medium to Low.
  • severity changed from Normal to Trivial.
  • milestone set to 1.2.6.

People who have changed their shell will most likely also figure this out by themselves. But sure, the message could possibly be changed to Run 'exec $shell' to blablabla where we fetch $shell from /etc/passwd.

08/01/08 20:02:18 changed by Jan-Nik

  • milestone changed from 1.2.6 to 1.3.

Milestone 1.2.6 deleted

08/02/08 08:36:07 changed by Jan-Nik

  • owner changed from taj to Jan-Nik.

basename $(cat /etc/passwd | grep ^$(whoami): | awk -F: '{print $7}')

Is this command ok to find out which shell the user is using?

08/02/08 11:38:19 changed by isak

Yeah, or maybe use the getent command, not sure: getent passwd `whoami` | awk -F: '{print $7}' Either is good I guess, maybe yours is even better since it doesn't rely on the getent program.

On thing to notice though: $7 may be empty, which, according to passwd(5), means that /bin/sh is the user's shell. If so, we should check if /bin/sh is bash (check symlink)

08/02/08 12:38:10 changed by Jan-Nik

  • status changed from new to assigned.

Ok, I fixed it in r2846, but I haven't tested it yet.

08/02/08 12:39:00 changed by Jan-Nik

Ups, i meant r2486

08/04/08 11:02:20 changed by Jan-Nik

  • status changed from assigned to closed.
  • resolution set to fixed.