<html><head></head><body><div dir="auto" style="direction: ltr; margin: 0; padding: 0; font-family: sans-serif; font-size: 11pt; color: black; ">1 I made the other merge requests by accident, I was learning how to do that. They can all be deleted or closed<br>
<br>
</div>
<div dir="auto" style="direction: ltr; margin: 0; padding: 0; font-family: sans-serif; font-size: 11pt; color: black; "><span id="OutlookSignature"><div dir="auto" style="direction: ltr; margin: 0; padding: 0; font-family: sans-serif; font-size: 11pt; color: black; ">thank you<br>
</div>
<div dir="auto" style="direction: ltr; margin: 0; padding: 0; font-family: sans-serif; font-size: 11pt; color: black; ">brian</div>
</span><br>
</div>
<br><br><br>
<div class="gmail_quote">On Sat, Feb 22, 2020 at 5:01 AM -0800, "Benoît Minisini" <span dir="ltr"><<a href="mailto:g4mba5@gmail.com" target="_blank">g4mba5@gmail.com</a>></span> wrote:<br>
<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">




<div dir="3D"ltr"">
<pre>Le 22/02/2020 à 01:06, Brian G a écrit :
> Fabien

> I attach a copy of Scripter which allows the inclusion of Libraries into 
> the compiled application.

> This is done with the use of the Lib directive eg. lib "company.diadic"

> After investigation about compiler, It will allow a full path name for 
> libraries. so I have changed it to to search 
> .local/gambas&version/lib/name&".gambas" and 
> /usr/lib/gambas&version/lib/name&".gambas", as well as just use the 
> provided name&".gambas"

> This seems to work and execute correctly.

> Would you be able to review this fits with your original design and if 
> so I will submit this change for merge.

> I am worried about dependencies although it seems to work in my testing 
> so far.

> Thank You
> Brian G


> ----[ http://gambaswiki.org/wiki/doc/netiquette ]----


Hi Brian,

1) Why did you make three merge requests? What should I do with them?

2) Why do you introduce a new keyword "LIB"? (That is not actually a new 
keyword until the compiler supports it). Why don't you use the "USE" 
keyword instead?

3) Why did you replace « Exec ["md5sum", $sPath] To $sMd5 » by « Shell 
"md5sum " & $sPath To $sMd5 » ? Which is by the way false as you don't 
quote the '$sPath' argument.

4) If you want to concatenate path elements, you have to use the '&/' 
operator.

5) Why did you call the option "--warn" and not "--warnings" as in the 
compiler?

Regards,

-- 
Benoît Minisini

----[ http://gambaswiki.org/wiki/doc/netiquette ]----
</pre>
</div>

</blockquote>
</div>
</body></html>