termux-packages icon indicating copy to clipboard operation
termux-packages copied to clipboard

plutolang: transfer as sub-package and prefer php scripts

Open 0komo opened this issue 1 year ago • 7 comments

  • Transfer plutolang as sub-package of libpluto, this follows the convention of liblua* and lua* package.
  • Prefer using PHP scripts found in scripts/ directory over Makefile as it's much stable and no patches required to build shared and static library.

0komo avatar Aug 25 '24 19:08 0komo

What would be the benefit of this change or why is this required?

Biswa96 avatar Aug 25 '24 19:08 Biswa96

Plutolang being a lua-like would loosely benefit from following the established structure of the liblua{51,52,53,54,jit} packages.

Needing PHP as the build system would be a minor pain in the ass. But if Komo feels like making those conformance changes I'm happy to merge them.

TomJo2000 avatar Aug 25 '24 20:08 TomJo2000

@TomJo2000 I feel like writing Makefile patch is much more pain in the ass than having to write a setup script for PHP :P

0komo avatar Aug 25 '24 20:08 0komo

I feel like writing Makefile patch is much more pain in the ass than having to write a setup script for PHP :P

That would be same as comparing screwdriver with hammer. Use the right tool for the right job.

Biswa96 avatar Aug 25 '24 20:08 Biswa96

I feel like writing Makefile patch is much more pain in the ass than having to write a setup script for PHP :P

That would be same as comparing screwdriver with hammer. Use the right tool for the right job.

IMO writing patches are more of a "Hey, apply this tape until it loosened", once the package got bumped, the patches may or may not work.

0komo avatar Aug 25 '24 20:08 0komo

And patching the PHP build scripts fixes this problem how exactly? I mean if it's what upstream recommends that's a good enough reason to use them, you don't have to come up with additional reasons.

TomJo2000 avatar Aug 26 '24 03:08 TomJo2000

And patching the PHP build scripts fixes this problem how exactly?

The PHP build scripts already added the thing you wanted, like building static and shared library. Unlike the Makefile approach where you need to patch the Makefile to also build a shared library and also use the Termux Builder's C{,XX,PP}FLAGS

I mean if it's what upstream recommends that's a good enough reason to use them, you don't have to come up with additional reasons.

They did recommend to use the PHP build scripts over the modified Makefile from Lua source.

0komo avatar Aug 26 '24 09:08 0komo