Skip to content

CFE-3288 Adjusted package_module and paths for termux platform#1804

Merged
nickanderson merged 1 commit intocfengine:masterfrom
craigcomstock:termux
Aug 3, 2020
Merged

CFE-3288 Adjusted package_module and paths for termux platform#1804
nickanderson merged 1 commit intocfengine:masterfrom
craigcomstock:termux

Conversation

@craigcomstock
Copy link
Copy Markdown
Contributor

@craigcomstock craigcomstock commented Jul 15, 2020

Packages promises only possibly supported back to 3.15.1 due to
need for interpreter attribute on package_module body and no
earlier version available for termux.

Paths added to support the interpreter path for packages.
Also several convenience paths added both for termux and generic unix:

  • paths.tmp_path, paths.bin_path, paths.var_path, paths.etc_path

Ticket: CFE-3288
Changelog: Title

@craigcomstock
Copy link
Copy Markdown
Contributor Author

ah, right, package_module body interpreter attribute is not supported in 3.10.x, will adjust accordingly with macros right?

Comment thread lib/paths.cf Outdated
Comment thread lib/packages.cf
Comment thread lib/paths.cf Outdated
Comment thread lib/packages.cf Outdated
Comment thread lib/packages.cf
Comment thread lib/packages.cf Outdated
Comment thread lib/paths.cf Outdated
Comment thread lib/paths.cf Outdated
Comment thread lib/paths.cf Outdated
Comment thread lib/paths.cf
Comment thread lib/paths.cf
@craigcomstock craigcomstock changed the title CFE-3288 Adjustements for termux non-privileged usage CFE-3288 Adjusted package_module and paths for termux platform Jul 31, 2020
@craigcomstock craigcomstock added the WIP Work in Progress label Jul 31, 2020
@craigcomstock
Copy link
Copy Markdown
Contributor Author

WIP, need to double-check/test paths.(all the commands) to make sure things are "all good".

Copy link
Copy Markdown
Member

@nickanderson nickanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems pretty ok to me. just a couple small commetns.

Comment thread lib/packages.cf Outdated
Comment thread lib/paths.cf
Comment thread lib/paths.cf
Packages promises only possibly supported back to 3.15.1 due to
need for interpreter attribute on package_module body and no
earlier version available for termux.

Paths added to support the interpreter path for packages.
Also several convenience paths added both for termux and generic unix:
- paths.tmp_path, paths.bin_path, paths.var_path, paths.etc_path

Ticket: CFE-3288
Changelog: Title
Copy link
Copy Markdown
Member

@nickanderson nickanderson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@craigcomstock craigcomstock removed the WIP Work in Progress label Jul 31, 2020
@nickanderson nickanderson merged commit 83c4e0c into cfengine:master Aug 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants