-
-
Notifications
You must be signed in to change notification settings - Fork 156
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
Build failed if --production or npm ci are used instead of development #416
Comments
Congratulations to ZeusLN for your 1000 downloads on Google Play! That made me try to reproduce the Play Store build, too and I ran into issues, too. I think though my main and/or only issue is that I fail to build the app without signing:
As I don't want to install it but verify the Play Store build, creating a signing setup should not be required. |
@Giszmo to build this app latest version 5.1 see Containerfile at:
Regarding the signing, just run the commands: cd android; \
keytool -genkey -alias zeus_alias -keystore app/zeus.pfx -storetype PKCS12 -keyalg RSA -keysize 4096 -storepass zeus_alias -keypass zeus_alias -validity 10000 -dname CN=IL; \
printf "\nMYAPP_RELEASE_STORE_FILE=zeus.pfx\nMYAPP_RELEASE_STORE_PASSWORD=zeus_alias\nMYAPP_RELEASE_KEY_PASSWORD=zeus_alias\nMYAPP_RELEASE_KEY_ALIAS=zeus_alias\n" >> gradle.properties; which will generate the required signing. |
I found that out eventually, too. It should either be documented in the build instructions or preferably not necessary if I don't want to release the app. That said, I did get a huge diff and ZeusLN at Twitter even announced that the app isn't reproducible right now. :( |
Opened new issue regarding reproducibility of APK latest version 0.6.0 with more details at: #898, this issue still apply as currently production NODE_ENV mode can't be used to compile the app. |
This is now solved with the addition of reproducible builds https://github.com/ZeusLN/zeus#reproducible-builds |
Describe the bug
Tried to build the app with using environment variable
NODE_ENV="production"
which is equivalent to adding--production
argument such asnpm install --production
command, the build failed with various errors, same happen whennpm ci
is used instead.building with
npm install --development
succeed.To Reproduce
Steps to reproduce the behavior:
git clone https://github.com/ZeusLN/zeus/; cd zeus;
npm install --production
cd android; ./gradlew assembleRelease
Expected behavior
it's also possible to build the app with
npm ci
andnpm install --production
which install less deps and is more secure due to it, also thenpm ci
which in addition to using the production argument, work differently in a way that help achieve reproducible builds.The errors are:
The text was updated successfully, but these errors were encountered: