[lokisetup] Problem with scripts not adding to installed_bytes

Stephane Peter megastep at megastep.org
Tue Jun 29 23:30:31 EDT 2004


OK, I committed patches to address these issues, mostly from Derek's 
work. The documentation is also updated to reflect the changes in the 
size syntax.

For the scripts returning odd error codes, I had never observed that 
behavior, and I use lots of scripts in my installers. So I'm wondering 
exactly what may be going on for you, the run_script() function is 
fairly simple. Though from looking at the code it could fail if your 
process was terminated by a signal.

I'm hesitant about adding error messages for failed scripts too, 
especially since copy.c is mostly shielded from the UI code (not that 
we couldn't do it anyway). Maybe just log a debug message that can be 
traced with setup -v 0 ...

On Jun 29, 2004, at 8:51 AM, Derek R. Gaston wrote:

> Ah, now I see - the size property is different for a script than it is 
> for an option - I don't think that's any good.  With both of my 
> updates (including https://bugzilla.icculus.org/show_bug.cgi?id=1828 ) 
> I think the script tag should now handle the same size format as 
> "option".
>
> Should we change the documentation?
>
> I understand the problem with the rc - but I don't understand why it's 
> giving incorrect output.  I even have all of my scripts "exit 0" at 
> the end - and they are still returning 1 - so I can only conclude that 
> something is broken.  I took a look at the code that actually calls 
> the script - and I don't think I'm qualified to make any changes in 
> that code.
>
> Maybe if someone else out there wants to give it a whirl and see if 
> they can get it to return 0 out of ANY script and then tell me how to 
> do it we can just revert that part - or like you said if rc!=0 maybe 
> throw an installation error.
>
> Derek
>
> Rob Crittenden wrote:
>
>> On Tue, 29 Jun 2004, Derek R. Gaston wrote:
>>
>>> Seriously: sz comes across as "10M" in my case - it just reads the 
>>> "size" property of the "script" tag.
>>>
>>
>> Right, the size property of the script tag is supposed to be in 
>> bytes, not
>> in shorthand. I can't control what is in the tag. It is good to use
>> the size_node() function in any case, I didn't see that function when 
>> I
>> added this.
>>
>> I think it is wrong to disregard the return value. I didn't have any 
>> problems
>> on my system with a bad return. One might argue that rc != 0 should 
>> return
>> an installation error and not continue at all. How else can the script
>> programmer return a useful error message?
>>
>> rob
>>
>>
>
>
--
Stéphane Peter
megastep at megastep.org




More information about the Lokisetup mailing list