 Notes |
(0010535)
yann (developer)
2013-08-16 17:15
|
I attached the modification I've made to make it work:
https://www.opencsw.org/mantis/file_download.php?file_id=67&type=bug [^] |
|
(0010536)
bonivart (manager)
2013-08-19 13:54
|
So you're basically adding another path to PATH and then removing all hardcoded paths? Looks very straightforward.
I can do a minor update with that. |
|
(0010560)
bonivart (manager)
2013-08-27 16:54
|
Did you test your patch on a normal Solaris system? I ran it on Solaris 10 and pkgadd/pkgrm are not found. You only added /opt/csw/bin to the path but some pkg tools are found in /usr/sbin.
Couldn't you accomplish the same thing by placing your pkg tools in the normal Solaris paths? |
|
(0010561)
yann (developer)
2013-08-27 21:27
|
You're right. I saw the standard /usr/bin:/bin paths and assumed that this was sufficient to have pkgutil find the native tools if they were present but I went too fast.
So I think the path modification should rather be:
-$ENV{'PATH'} = '/bin:/usr/bin';
+$ENV{'PATH'} = '/usr/sbin:/bin:/usr/bin:/opt/csw/bin';
To answer your question: no I can't, smartos /usr partition is readonly and I am using a cloud environment where I don't have control on the OS. |
|
(0010563)
yann (developer)
2013-08-29 23:44
|
I updated the patch accordingly. I tested on Solaris 10 and it works fine.
https://www.opencsw.org/mantis/file_download.php?file_id=68&type=bug [^]
Sorry for the previous defective patch. |
|
(0010567)
bonivart (manager)
2013-09-02 13:46
|
There's packages here: http://buildfarm.opencsw.org/experimental.html#bonivart [^]
Even though it's a simple change I would like the two of us to try it some before I release it officially. |
|
(0010600)
yann (developer)
2013-10-06 12:08
|
For me no problem so far since I installed this version (I installed a few days after your last message). |
|
(0010636)
yann (developer)
2013-11-09 19:00
|
Hi Peter,
Still no problem with this version, do you think you will release it shortly ?
Yann |
|
(0010639)
bonivart (manager)
2013-11-13 10:35
|
I forgot all about it, sorry. It has now been released. |
|
no_hardcoded_paths.patch [^] (4,304 bytes) 2013-08-16 17:15
