[PACKAGED] bootinfoscript - Is it OK?

[sling-shot]

Secondary to a package request from here - http://www.pclinuxos.com/forum/index.php/topic,113128.0.html
It looks interesting to have and might help forum members gather information while troubleshooting fellow members' problems with booting etc.

Summary :
Boot Info Script is a bash script which searches all hard drives attached
to the computer for information related to booting. Its primary use is for
troubleshooting booting problems.

URL :
http://sourceforge.net/projects/bootinfoscript/

SPEC - http://dl.dropbox.com/u/37931428/bootinfoscript/slingshot-bootinfoscript-01.spec
LOG - http://dl.dropbox.com/u/37931428/bootinfoscript/slingshot-bootinfoscript-01.spec.log
RPM - http://dl.dropbox.com/u/37931428/bootinfoscript/bootinfoscript-0.61-1slingshot2013.noarch.rpm
SRPM - http://dl.dropbox.com/u/37931428/bootinfoscript/bootinfoscript-0.61-1slingshot2013.src.rpm

[ghostbunny]

Hi sling-shot,

i read a bit in the script and it needs root rights so it should be installed to /usr/sbin not /usr/bin

it uses wget or curl so you should add wget or curl as Requires

the rest of required things seems to be core stuff which should be installed everywhere

ah and your defined version seems to miss a dot

except of this three little things it looks very ok for me

regards
ghostbunny

[sling-shot]

Hi sling-shot,

i read a bit in the script and it needs root rights so it should be installed to /usr/sbin not /usr/bin

it uses wget or curl so you should add wget or curl as Requires

the rest of required things seems to be core stuff which should be installed everywhere

ah and your defined version seems to miss a dot

except of this three little things it looks very ok for me

regards
ghostbunny

That was new to me. Can I take it in general that anything that needs to run as root should go into /usr/sbin?

Regarding wget/curl - are they not for updating the application itself? It says updating the application does not require root. Anyway as we do not prefer applications that auto-update, it may not be important I guess. Unless I am missing something else here.

The version dot. It confused the hell out of me. It looks like the actual directory inside the source tarball does not have the dot in its name. Although the version is properly named the directory is not. Either I had to open the tarball, rename directory and then recompress or I could adjust the .spec. I went for the latter. Is it wrong to do that?

[ghostbunny]

That was new to me. Can I take it in general that anything that needs to run as root should go into /usr/sbin?

i think so.

Regarding wget/curl - are they not for updating the application itself? It says updating the application does not require root. Anyway as we do not prefer applications that auto-update, it may not be important I guess. Unless I am missing something else here.

How do you want to get writing access to /usr/(s)bin without root rights?

The version dot. It confused the hell out of me. It looks like the actual directory inside the source tarball does not have the dot in its name. Although the version is properly named the directory is not. Either I had to open the tarball, rename directory and then recompress or I could adjust the .spec. I went for the latter. Is it wrong to do that?

it is not necessary to rename the folder just fix the setup line

Code: [Select]

%setup -q -n %name-<version without dot>


EDIT:
ok when you update the program it saves the new script in the /home/<user> but you also need root rights to execute it. bit senseless for me :/

[sling-shot]

I had to fix the Source: line too in order to get it to build. I have updated the DropBox links above.

[ghostbunny]

a little tip for the laziness

add a %define sversion 061 and replace the occurrences of 061 in the spec with %sversion so you have to update just this line and not the two others .

i think also it is ready for submitting to the packagers db.

regards
ghostbunny

[sling-shot]

Done.