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

Include further helper scripts in deb that main ATLAS script sources #93

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

AtlasMUC
Copy link

The main ATLAS script is sourcing further helper scripts, directly or indirectly, that are currently not included in the deb package built by the make-deb script. Update make-deb to have them included.
The issue raised in PR #88 currently hides the fact that those scripts are missing, as the non-supported source command triggers an error message by the shell, but the ATLAS script itself still continues to work, as well as the core probe functionality, as far as I can see.
However, with the (valid) changes proposed in PR #88, the . trying to source non-existing helper scripts leads to complete abortion of the ATLAS script, and thus failure of the probe.

The main ATLAS script is sourcing further helper scripts, directly or indirectly, that are currently not included in the deb package built by the make-deb script. Update make-deb to have them included.
@AtlasMUC
Copy link
Author

This is kind of for Debian what #79 is/was for OpenWrt, with a similar solution.

@michel-stam
Copy link
Contributor

Hi @AtlasMUC

As of next release there will be a new build for DEB files available, Debian 11/12 will be supported on x86_64.

Please be patient its being worked on, the new version will use dpkg-buildpackage, and not make-deb anymore.

Regards,

Michel

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.

2 participants