MantisBT - Buildsystem | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0002546 | Buildsystem | [All Projects] General | public | 2017-06-04 18:05 | 2018-04-21 09:27 |
Reporter | Michael Siegel | ||||
---|---|---|---|---|---|
Assigned To | |||||
Priority | low | Severity | tweak | Reproducibility | always |
Status | closed | Resolution | won't fix | ||
Platform | GNU/Linux | OS | Devuan | OS Version | 1.0.0 (Jessie) |
Fixed in CI build # | |||||
Reported in CI build # | |||||
Summary | 0002546: Replace "sudo" with "su -c" in env.sh | ||||
Description | The function "ns-apt-get-install" inside the temporary env.sh used in building NetSurf from source on GNU/Linux uses the command sudo apt-get install $(echo ${NS_DEV_DEB} ${NS_TOOL_DEB} ${NS_GTK_DEB}) to install packages needed to build and run NetSurf on Devuan/Debian. This is not ideal since "sudo" cannot be expected to be installed on such a system. I therefore suggest replacing that command with su -c "apt-get install $(echo ${NS_DEV_DEB} ${NS_TOOL_DEB} ${NS_GTK_DEB})" You could even make it su -c "apt-get install $(echo ${NS_DEV_DEB} ${NS_TOOL_DEB} ${NS_GTK_DEB}) --no-install-recommends" to reduce the number of packages being pulled. I've used this command in building NetSurf and it doesn't seem to have caused any harm. It seems, however, that using "su" instead of "sudo" will break things for Ubuntu distributions, since "by default, the root account password is locked in Ubuntu" (https://help.ubuntu.com/community/RootSudo). Still, these systems can be easily configured to allow the use of "su". See https://askubuntu.com/questions/44418/how-to-enable-root-login . Btw, the above is probably also a problem in "ns-yum-install". | ||||
Steps To Reproduce | Do the following on a Devuan/Debian system that doesn't have "sudo": $ wget http://git.netsurf-browser.org/netsurf.git/plain/Docs/env.sh $ source env.sh $ ns-package-install | ||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files |
Notes | |||||
|
|||||
|
|
||||
|
|||||
|
|
Issue History | |||||
Date Modified | Username | Field | Change | ||
---|---|---|---|---|---|
2017-06-04 18:05 | Michael Siegel | New Issue | |||
2017-06-04 18:18 | Michael Siegel | Note Added: 0001565 | |||
2018-04-21 09:27 | Daniel Silverstone | Status | new => closed | ||
2018-04-21 09:27 | Daniel Silverstone | Resolution | open => won't fix | ||
2018-04-21 09:27 | Daniel Silverstone | Note Added: 0001754 |