[Bug 442266] Review Request: embryo - Easy to use library for running compiled PAWN programs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: embryo - Easy to use library for running compiled PAWN programs


https://bugzilla.redhat.com/show_bug.cgi?id=442266





------- Additional Comments From pertusus@xxxxxxx  2008-05-06 03:23 EST -------
(In reply to comment #6)
> Embryo is for scripting existing applications, similar to LUA, and embryo 
> scripts are usually distributed as source code. It's not the case of C 
> compiler, such scripts are compiled and ran by regular users.

> LUA ships compiler in one package with libs.

Indeed, but in the case of embryo, it seems to be much more common
to use directly the libs. And e17 packages depending on embryo
only use the libs, so it seems to me that putting the interpreter in 
its own pacage (with name embryo) would be better, in order not to 
install the interpretr if not needed. Moreover the interpreter
(in contrast with the one embedded in the libs) seems to be less
well maintained (reading the README). Also, unless I am missing something
the %description is mostly for the -libs package, the interpreter has
still not been that much modified.

Now regarding the segfault, in my opinion it is much better to do a 
patch (and submit it upstream). A sed substitution will silently fail 
and in general is less readable. In the end it depends on the 
packager, but in my opinion the change in argv should be a patch, 
while the 2 other sed substitutions are right.

As a side note, this segfault is also an indication that the 
interpretor is not widely used.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]