diff -u /etc/rc.d/rc.mM /etc/rc.d/rc.M.new a maybe minor nit ...

babydr · 03-27-2022, 08:15 PM

Hmm , relative references to executables in the start up ...

Code:

-# Start Samba (a file/print server for Win95/NT machines).
+# Start the mail server. Try the rc.sendmail script first, then try rc.postfix.
+if [ -x /etc/rc.d/rc.sendmail -a -x usr/sbin/makemap ]; then
+  /etc/rc.d/rc.sendmail start
+elif [ -x /etc/rc.d/rc.postfix -a -x usr/sbin/postdrop ]; then
+  /etc/rc.d/rc.postfix start
+fi

bassmadrigal · 03-28-2022, 05:01 PM

It took me a minute to realize what was going on with this thread.

For any confused by it like I was, babydr is commenting on Pat's use of relative directories for calling usr/sbin/makemap and usr/sbin/postdrop instead of /usr/sbin/makemap and /usr/sbin/postdrop.

I'm not sure what the $PWD is in the starting scripts. If it is /, then the scripts should still work, although, it's not a bad idea to change it to use absolute paths.

Mechanikx · 03-28-2022, 05:22 PM

Quote:

Originally Posted by bassmadrigal

It took me a minute to realize what was going on with this thread.

For any confused by it like I was, babydr is commenting on Pat's use of relative directories for calling usr/sbin/makemap and usr/sbin/postdrop instead of /usr/sbin/makemap and /usr/sbin/postdrop.

I'm not sure what the $PWD is in the starting scripts. If it is /, then the scripts should still work, although, it's not a bad idea to change it to use absolute paths.

I was curious so I put in an

Code:

echo "$PWD"

right after the line "Going multiuser" and at the very end of the script and it is '/'

bassmadrigal · 03-28-2022, 05:36 PM

Quote:

Originally Posted by Mechanikx

I was curious so I put in an

Code:

echo "$PWD"

right after the line "Going multiuser" and at the very end of the script and it is '/'

That's good it isn't actually broken using relative paths to those commands.

Thanks for checking, I was thinking of doing that when I get home.

Mechanikx · 03-28-2022, 06:01 PM

Quote:

Originally Posted by bassmadrigal

That's good it isn't actually broken using relative paths to those commands.

Thanks for checking, I was thinking of doing that when I get home.

No worries.

I looked through the rest of the script and there aren't any more relative paths that I can see. So, if it is just a typo it's a sneaky one because since you're in / it still works.

bassmadrigal · 03-28-2022, 07:45 PM

Quote:

Originally Posted by Mechanikx

No worries.

I looked through the rest of the script and there aren't any more relative paths that I can see. So, if it is just a typo it's a sneaky one because since you're in / it still works.

Yep, I checked both rc.M and rc.S for the same

Mechanikx · 03-28-2022, 08:02 PM

Quote:

Originally Posted by bassmadrigal

Yep, I checked both rc.M and rc.S for the same

I guess we'll see in the changelog if it's an accident or on purpose. But I can't think of a reason why a relative path would be used.

drumz · 03-29-2022, 03:35 PM

Quote:

Originally Posted by Mechanikx

I guess we'll see in the changelog if it's an accident or on purpose. But I can't think of a reason why a relative path would be used.

Code:

Tue Mar 29 20:16:24 UTC 2022
a/sysvinit-scripts-15.0-noarch-9.txz:  Rebuilt.
  rc.M: Fix cosmetic mistake where a couple of paths were relative instead of
  absolute. Thanks to babydr.

johnny23 · 03-30-2022, 06:39 PM

Quote:

Originally Posted by drumz

Code:

Tue Mar 29 20:16:24 UTC 2022
a/sysvinit-scripts-15.0-noarch-9.txz:  Rebuilt.
  rc.M: Fix cosmetic mistake where a couple of paths were relative instead of
  absolute. Thanks to babydr.

It's in the -current changelog but not the -15.0 ... it should be in both.

Mechanikx · 03-30-2022, 07:06 PM

Quote:

Originally Posted by johnny23

It's in the -current changelog but not the -15.0 ... it should be in both.

I don't think it will be in 15.0 since it only receives security updates and this was a cosmetic fix.