[Bug 1270554] Review Request: haxe - Multi-target universal programming language

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

 



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



--- Comment #11 from Andy Li <andy@xxxxxxxxxxxxxx> ---
I've revised the package as suggested. The updated files:
Spec URL: https://dl.dropboxusercontent.com/u/2661116/fedora/haxe/haxe.spec
SRPM URL:
https://dl.dropboxusercontent.com/u/2661116/fedora/haxe/haxe-3.2.1-1.fc24.src.rpm

Summary of changes:
 * Package the latest upstream version (3.2.1).
 * Introduce subpackage, haxe-stdlib, for the Haxe standard library to be
installed in /usr/share/haxe/.
 * Fix permission of "haxe".
 * Add license breakdown in spec. Also requested license fix in
https://github.com/HaxeFoundation/haxe/issues/4602.
 * Fix srpm-spec mismatched versions. Those were introduced by the openSUSE
build service I was using. This time I used my machine to build and uploaded to
dropbox instead.
 * Use help2man to generate man pages for haxe and haxelib.
 * Use pushd/popd instead of cd.

Things I'm not sure:
 * I'm not sure about the install error. I've tested with fedora-review, and
there is indeed an error saying "Error: cannot install both
haxe-debuginfo-3.2.1-1.fc24.x86_64 and haxe-debuginfo-3.2.1-1.fc24.x86_64" but
that's totally weird to me. Full log can be found at
https://dl.dropboxusercontent.com/u/2661116/fedora/haxe/root.log. I can
manually install the rpms without error by `sudo rpm -i *.rpm`.
 * I'm not sure whether I have to add "Requires" to the debuginfo package. I
browse around the Internet and couldn't find an example of that.
 * I don't think there is anything that fits into a haxe-devel package. There
is no such thing as "to develop software that uses haxe", because haxe itself
is a dev tool and it is not designed to be linkable and there is no plug-in
system.
 * The remaining "incorrect-fsf-address" are fixed in upstream. They were fixed
by the patch in the first version of this package. I haven't included the patch
in the revised package.

Thank you and best regards,
Andy

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review




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