-
-
Notifications
You must be signed in to change notification settings - Fork 495
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
DietPi-Software | MineOS installation breaks on npm #1880
Comments
@MichaIng Reach out to the author |
Thanks |
Retest on VM Jessie: Terminal output
end of debug.log
Switch to Node v8:
Using alternative method to install NodeJS via repo:
|
Restart from scratch, using official Node v8 install method directly:
🈯️ Install succeeds. Above install method creates node binary at expected location, thus @Fourdee
With Java we could btw. do the same, as Java 10 is available. |
Yep I agree 👍 , then offer alternative versions at a later date in the future? Only issue is we'd need to fully this test with all node pre-req programs, and, somehow patch existing users down to this version (we could probably uninstall then install again I believe for this). |
@Fourdee We could call the new install NodeJS legacy e.g., then it's clear that it should only be seen as a compatibility fallback. I will create a PR to test: https://github.com/Fourdee/DietPi/pull/1936 |
Okay, actually the Koel solution works for MineOS as well. Node 8 just needs to be installed directly instead of failed Node 10 install attempt 😄. Will test on all Distros: €: Okay commit added: Fourdee@8289982 |
+ DietPi-Software | MineOS: Install Node 8 for compatibility reasons: https://github.com/Fourdee/DietPi/issues/1880 + DietPi-Software | NodeJS: Cleaning up NodeJS by first uninstalling global modules and remove possible module directories
ARMv7 Stretch, MineOS + NetData:
Doesn't support symlinks? Ah 🈴
🈺 Christ, 2nd test is fine, weird. We have no G_HW_ARCH usage here, so appears the nodejs install script is to blame.
|
@Fourdee Strange, but Node 10 is the correct ARMv7 version? |
Yep:
|
+ MineOS install reordering/cleanup: https://github.com/Fourdee/DietPi/issues/1880
Okay, retested successfully here. Hmm Node installs/builds stay a bid unreliable, but it's the best we can do? €: |
Yep, outside our control. Just trying to run MineOS as non-root, will send commit if successful. supervisor process reports
MineOS fails to load:
So we can assume that under the hood of supervisor, mineOS is running as non-root user. However, web login fails and renders a Or
is somehow different to Nope, must run as |
+ finalise mineos: https://github.com/Fourdee/DietPi/issues/1880
🈯️ Test install passed. Completed. |
**v6.12** (19/07/18) **Changes / Improvements / Optimizations:** DietPi-Drive_Manager | Samba/CIFS mounting: Now automatically uses the highest available CIFS version supported on client and server: https://github.com/Fourdee/DietPi/issues/1893#issuecomment-403034799 DietPi-Software | Jackett: Now runs as its own user, and, from the /opt/jackett directory, for new installations only. Many thanks to @userdeveloper98 for contributing this improvement: https://github.com/Fourdee/DietPi/pull/1895 DietPi-Software | MiniDLNA: Now uses a SystemD service, also updates its library during service start. DietPi-Software | JRiver: Removed and no longer available for installation: https://github.com/Fourdee/DietPi/issues/1080#issuecomment-403489246 DietPi-Software | Various titles: Now run under their own system user account, with limited permissions (previously root): https://github.com/Fourdee/DietPi/issues/1877#issuecomment-403298679 DietPi-Software | SABnzbd: Language packs are now installed by default: https://github.com/Fourdee/DietPi/issues/1917#issue-340631943 DietPi-RAMlog | Increased the max size of /var/log to 50MB by default (previously 20MB). This should prevent 0 free space errors for excessive log file usage. Pi-Hole max logfile size is now 50MB/3 (16MB~) for high usage with daily stats support: pi-hole/pi-hole#2270 (comment) | https://github.com/Fourdee/DietPi/issues/1923 DietPi-RAMlog | Service is now disabled when RAMlog mode is not selected: https://github.com/Fourdee/DietPi/issues/1924 **Bug Fixes:** General | Resolved an issue where cron jobs, containing DietPi scripts, failed: https://github.com/Fourdee/DietPi/issues/1923 General | Resolved an issue on ARM64 + Jessie with APT, due to debian-security removing suppport and packages for those devices. If you experience this issue, and are unable to update DietPi, please see : https://github.com/Fourdee/DietPi/issues/1915 General | Resolved an issue where NFSv3 network drives could not be mounted: https://github.com/Fourdee/DietPi/issues/1898 DietPi-Config | ASUS TB: Resolved loss of WiFi device after a reboot: https://github.com/Fourdee/DietPi/issues/1760 DietPi-Drive_Manager | Resolved an issue where the program could remove a non-empty directory in rare situations. DietPi-Software | Resolved a potential Mono instability issue with Radarr, Sonarr and Jackett, due to using '--optimize=all --server'. This has now been removed for new installations. Many thanks to @hellfirehd for debugging/testing and @Taloth for dev insights: https://github.com/Fourdee/DietPi/issues/1896 DietPi-Software | Mono: Temp mono files are now cleared from memory once installed, preventing out of memory errors for additional software installs afterwards: https://github.com/Fourdee/DietPi/issues/1877#issuecomment-403856446 DietPi-Software | Xserver: Resolved rarely occuring uninstall issus by not purging dependencies, but leaving them for autoremove: https://github.com/Fourdee/DietPi/issues/1921 DietPi-Software | MineOS: Resolved failed installation due to incompatibilities with nodejs v10. v8 is now installed: https://github.com/Fourdee/DietPi/issues/1880 DietPi-Update | Resolved an issue where incorrect version would be displayed, once update was completed. This is due to '| tee' on a function, making var changes local: https://github.com/Fourdee/DietPi/issues/1877#issuecomment-403866204
I'm having this same issue when attempting to install MineOS on a RockPro64 (aarch64) with Dietpi v6.21.1, albeit with node v8.15.0 instead of v8.11.3. Could you please expand on how you were able to get n to install the correct architecture here:
Update: I was able to get MineOS installed by reinstalling Node via dietpi-software, manually installing n with |
@DeathIsUnknown |
I believe so, yes. Even now, if I try
According to this issue's comment, this error crops up when n can't find a corresponding pre-built binary from nodejs. However, arm64 binaries are available, and specifying
I am assuming that, as Fourdee found in his test, n is still attempting to target x86 (of which Node has dropped Linux support since v10) on my ARMv8 device - unfortunately I don't have other hardware to confirm if it's a common issue across similar devices. |
@DeathIsUnknown
Now that you say it I remember this issue, ah lol as of this topic 😄. AFAIK it was indeed only for ARMv8 devices, but perhaps to be failsafe we should simply always add the |
Looks like a pull request was made to add support for ARMv8 back in 2017, but it still hasn't been approved. tj/n#448 |
@DeathIsUnknown Lets get this fixed for all PR up and merged: https://github.com/Fourdee/DietPi/pull/2562/files Many thanks for reporting and researching this 👍!! I mark the issue as closed again, but feel free to use, if required. |
Bit of a follow-up, |
@DeathIsUnknown
|
@MichaIng My SBC is offline at the moment until I figure out a kernel issue, so I won't be able to test on an ARM64 platform. Sorry! |
Creating a bug report/issue:
Required Information:
v6.9
Stretch
4.9.88
Additional Information (if applicable):
Steps to reproduce:
Expected behaviour:
npm --install
runs through without error.Actual behaviour:
npm --install
throws a lot of warnings and finally error messages.Extra details:
npm install log
npm install -g n && n 8
The text was updated successfully, but these errors were encountered: