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

bump node version to 16 #21

Merged
merged 18 commits into from
Mar 2, 2023
Merged

bump node version to 16 #21

merged 18 commits into from
Mar 2, 2023

Conversation

muelli
Copy link
Collaborator

@muelli muelli commented Apr 1, 2022

No description provided.

@flathubbot
Copy link
Contributor

Started test build 84333

@flathubbot
Copy link
Contributor

Build 84333 failed

muelli added 3 commits March 1, 2023 15:31
with flatpak-node-generator as of
c278045d7cc73437945b3dfb919fc7aea5d43cb5

python3 -m flatpak_node_generator yarn ~/vcs/lossless-cut/yarn.lock
--output ~/vcs/no.mifi.losslesscut/generated-sources.json

This also requires a change in the build definition to include
the yarn configuration.
@flathubbot
Copy link
Contributor

Started test build 25868

Yarn is built in now, isn't it?
@flathubbot
Copy link
Contributor

Build 25868 failed

@flathubbot
Copy link
Contributor

Started test build 25870

@flathubbot
Copy link
Contributor

Started test build 25873

@flathubbot
Copy link
Contributor

Build 25873 failed

@flathubbot
Copy link
Contributor

Build 25870 failed

@flathubbot
Copy link
Contributor

Started test build 25901

@flathubbot
Copy link
Contributor

Build 25901 failed

weird.
Somehow, the yarn.lock file in the lossless-cut repository got changed.
Now it corresponds to the 3.52.0 tag.
dd1b44f2d3d645056df78175b421aff414b103e2
@flathubbot
Copy link
Contributor

Started test build 25912

@flathubbot
Copy link
Contributor

Started test build 25915

@flathubbot
Copy link
Contributor

Build 25912 failed

@flathubbot
Copy link
Contributor

Build 25915 failed

@catsout
Copy link

catsout commented Mar 2, 2023

@flathubbot
Copy link
Contributor

Started test build 25922

@flathubbot
Copy link
Contributor

Build 25922 failed

I wonder why this is necessary, though, because we're calling
pack-linux and there is a "prepack-linux" script defined.
Let's see whether this works.
@flathubbot
Copy link
Contributor

Started test build 25930

@flathubbot
Copy link
Contributor

Build 25930 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/8589/no.mifi.losslesscut.flatpakref

@muelli
Copy link
Collaborator Author

muelli commented Mar 2, 2023

The app works. Thanks @catsout.

But there's a big ASAR file, despite electron-builder complaining about the lack of ASAR:

Running: yarn run pack-linux dir
FB: Running 'flatpak build --die-with-parent --env=FLATPAK_BUILDER_BUILDDIR=/run/build/lossless-cut --nofilesystem=host:reset --filesystem=/srv/buildbot/worker/build-x86_64-2/build/.flatpak-builder/build/lossless-cut-1 --bind-mount=/run/build/lossless-cut=/srv/buildbot/worker/build-x86_64-2/build/.flatpak-builder/build/lossless-cut-1 --build-dir=/run/build/lossless-cut --bind-mount=/run/ccache=/srv/buildbot/worker/build-x86_64-2/build/.flatpak-builder/ccache --unshare=network --env=SOURCE_DATE_EPOCH=1677771815 '--env=CFLAGS=-O2 -g -pipe -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions -fstack-protector-strong -grecord-gcc-switches -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -fno-omit-frame-pointer ' '--env=CXXFLAGS=-O2 -g -pipe -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions -fstack-protector-strong -grecord-gcc-switches -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -fno-omit-frame-pointer ' '--env=LDFLAGS=-L/app/lib -Wl,-z,relro,-z,now -Wl,--as-needed ' --env=XDG_CACHE_HOME=/run/build/lossless-cut/flatpak-node/cache --env=npm_config_nodedir=/usr/lib/sdk/node18 --env=NPM_CONFIG_LOGLEVEL=info --env=CCACHE_DIR=/run/ccache/disabled --env=PATH=/app/bin:/usr/bin:/usr/lib/sdk/node18/bin --env=LD_LIBRARY_PATH=/app/lib --env=PKG_CONFIG_PATH=/app/lib/pkgconfig:/app/share/pkgconfig:/usr/lib/pkgconfig:/usr/share/pkgconfig --env=FLATPAK_BUILDER_N_JOBS=64 /srv/buildbot/worker/build-x86_64-2/build/.flatpak-builder/rofiles/rofiles-G97wXT /bin/sh -c 'yarn run pack-linux dir'' on host
  • electron-builder  version=23.6.0 os=5.15.0-47-generic
  • loaded configuration  file=package.json ("build" field)
  • writing effective config  file=dist/builder-effective-config.yaml
  • packaging       platform=linux arch=x64 electron=23.1.0 appOutDir=dist/linux-unpacked
  • asar usage is disabled — this is strongly not recommended  solution=enable asar and use asarUnpack to unpack files that must be externally available
  • asar usage is disabled — this is strongly not recommended  solution=enable asar and use asarUnpack to unpack files that must be externally available
FB: host_command_exited_cb 1419254 0

I wonder how to make it not produce an ASAR file.

@catsout why does this work for you?
https://github.com/flathub/net.cozic.joplin_desktop/blob/8aad869adfc541c07a9284aad33aad59165b1a33/net.cozic.joplin_desktop.yml#L82
The package.json does not define a "dist" target.

@muelli muelli mentioned this pull request Mar 2, 2023
@muelli
Copy link
Collaborator Author

muelli commented Mar 2, 2023

hm. Thanks. It's gone from here, though: https://github.com/laurent22/joplin/blob/dev/package.json

What happens to the arguments? Are they simply appended to the target? i.e. yarn run electronRebuild && npx electron-builder --x64 --config.asar=false --linux --dir?

@flathubbot
Copy link
Contributor

Started test build 25934

@flathubbot
Copy link
Contributor

Build 25934 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/8593/no.mifi.losslesscut.flatpakref

@muelli
Copy link
Collaborator Author

muelli commented Mar 2, 2023

looks better now.

I think the modern generators take care of those.
@flathubbot
Copy link
Contributor

Started test build 25944

we call the packing with the --config.asar=false flag which makes it work better.
@flathubbot
Copy link
Contributor

Started test build 25945

the file is automatically generated now.
@flathubbot
Copy link
Contributor

Started test build 25946

@flathubbot
Copy link
Contributor

Build 25944 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/8603/no.mifi.losslesscut.flatpakref

@flathubbot
Copy link
Contributor

Build 25946 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/8605/no.mifi.losslesscut.flatpakref

@flathubbot
Copy link
Contributor

Build 25945 successful
To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/8604/no.mifi.losslesscut.flatpakref

@muelli muelli merged commit 2d1b26e into master Mar 2, 2023
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.

3 participants