Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add perl-parse-yapp #5417

Closed
wants to merge 1 commit into from
Closed

Conversation

qwerty12
Copy link
Contributor

@qwerty12 qwerty12 commented Jun 17, 2020

First of two build-deps needed for the Samba package I'll be making a PR for soon in unstable-packages.

Some things of note:

  1. Makefile.PL is ran by the Ubuntu's host Perl. Currently, not a problem: the files produced are identical to those produced when built directly on my phone.

  2. This module's copyright notice is "You may use and distribute them under the terms of either the GNU General Public License or the Artistic License, as specified in the Perl README file.". I've interpreted that to mean GPL v1 / Artistic License v1, and it's the former licence that I've elected to have installed by this package. Fedora has this to say on the subject of Perl licenses.

EDIT: Now that I think of it, getting the Samba build system to see this module will be a fun endeavour :) Willing to try, though.

@ghost
Copy link

ghost commented Jun 17, 2020

One question: is perl-parse-yapp used during Samba run time or just to build package ?

@qwerty12
Copy link
Contributor Author

Just to build the package, @xeffyr

@ghost
Copy link

ghost commented Jun 17, 2020

Then it should be installed to host by https://github.com/termux/termux-packages/blob/master/scripts/setup-ubuntu.sh.

@qwerty12
Copy link
Contributor Author

Thanks, that was short-sighted of me.

@qwerty12 qwerty12 closed this Jun 17, 2020
ghost pushed a commit that referenced this pull request Jun 17, 2020
@ghost
Copy link

ghost commented Jun 17, 2020

Added to Ubuntu/Docker setup script in 1d67012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant