NWjs versioning depending on OS

[nerdCopter]

• NWjs versioning depending on OS; this is not preferred, but maybe necessary
• fixes Cannot open BBE on Mac Intel 64 since 687 since bumping NWJS and FFMPEG #696
• proof-of-concept; DRAFT while in discussion/consideration
• this PR will re-test historical failure of GitHub action for NWjs 0.72.0
• win32 was not part of the issue, but added it as such is known to work.

[haslinghuis]

Yeah - working again - no 504

[nerdCopter]

• i don't like this solution; however, it seems to work.
• NW.js and ffmpeg may be un-syncronized with this.
• @ctzsnooze , can you test the executable-package created by this PR?

[sonarqubecloud]

Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[haslinghuis]

One issue might be with the ffmpeg libraries - not sure if we could retrieve the exact OSX version used - so to minimize impact of this change only applying to older versions.

[haslinghuis]

Maybe https://www.npmjs.com/package/get-osx-version

[github-actions]

Do you want to test this code? Here you have an automated build:
Betaflight-Blackbox-Explorer-Linux
Betaflight-Blackbox-Explorer-macOS
Betaflight-Blackbox-Explorer-Windows
WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

[nerdCopter]

i found some issue with any logic for when using gulpfile to retrieve the OS version (kernel). it is built on modern OSX (kernel), therefore will still choose modern NWjs, and thusly not be backward compatible.
i tried os.release().startsWith('1') , where kernel is 1x.x.x for any OSX 10.x. kernel 2x.x.x for any OSX 11.x

[haslinghuis]

Can we have a macOSDeprecated strategy in the workflow matrix?

blackbox-log-viewer/.github/workflows/ci.yml

Lines 16 to 33 in a11ed06

	jobs:
	build:
	name: Build (${{ matrix.name }})
	runs-on: ${{ matrix.os }}
	strategy:
	matrix:
	include:
	- name: Linux
	os: ubuntu-22.04
	releaseArgs: --linux64
	- name: macOS
	os: macos-12
	releaseArgs: --osx64
	- name: Windows
	os: windows-2022
	releaseArgs: --win64

[haslinghuis]

But guess soon we won't be able to do so:

[nerdCopter]

the future is now 😄 ☹️

[nerdCopter]

saw some NWjs issue tracker comments alluding to custom universal packages , but this is just more effort / maintenance :(

finding some more stuff.
maybe [only] a newer nw-builder is required: nwutils/nw-builder#783
basically this nwjs/nw.js#8009 lead me down a rabbit-hole.

[ctzsnooze]

Solves the problem on my older Mac. So long as newer Macs are OK, I'm happy!
Thank you!

[haslinghuis]

Yes needs testing on most recent Macs - otherwise we might have two implement separate builds for new and old Macs

[ikherty]

looks and works as usual, no problems noticed. MacBook Air M2 13.6" 512/16 GB, macOS Sonoma 14.3

Do you want to test this code? Here you have an automated build: Betaflight-Blackbox-Explorer-macOS WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

[haslinghuis]

@ikherty did you also test ffmpeg part?

[ikherty]

@ikherty did you also test ffmpeg part?

oh, sorry, I didn't try to play the video file. now I tried to open it, but the app closes immediately after selecting the file. I tried with different files, the behaviour is the same..

Issues found

[nerdCopter]

thank you @ikherty . sorry @haslinghuis @ctzsnooze , i don't know a way to import different ffmpeg versions depending on NWjs version ☹️ .

maybe a BASH script at build-time ☹️ .

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs within a week.

[github-actions]

Pull request closed automatically as inactive.