-
Notifications
You must be signed in to change notification settings - Fork 13
Update Joplin to 3.4.12 #285
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
base: master
Are you sure you want to change the base?
Conversation
|
🚧 Test build enqueued. |
|
🚧 Started test build. |
|
❌ Test build failed. Help
|
|
You have to update the checksum too. |
Ah, I forgot about that. I just quickly made this PR. I'll do that soon. Thanks for the catch! 😄 |
cb74db4 to
2756287
Compare
|
🚧 Test build enqueued. |
|
🚧 Started test build. |
|
❌ Test build failed. Help
|
|
Hi,
Do you already know what is causing the error? I'd love to get a new release, because a plugin I am trying to use requires it. BTW: Would it even be possible to merge the PR without @catsout, who has done it in the past, but is not as active as before in this project? |
On your end, you need to install the appropriate runtimes. Do On the builder's end, I think it's because I haven't updated the dependencies here. Once I figure that out (provided I have the time to do so because I'm a little busy right now), I'll update this PR accordingly.
I'm not sure if there is, but hopefully they see this PR and merge it once it's ready! 😄 |
|
For the yarnpkg-related build failure (line 1442 in the build log): This seems to be related to this line in a custom Yarn plugin: // Line 133 in flatpak_node_generator/flatpak-yarn.js
const lockfilePath = ppath.join(this.context.cwd, configuration.get(`lockfileFilename`));Joplin 3.4.x uses Yarn v4, which no longer includes the Note: The Edit: See this comment for a possible workaround. |
2756287 to
926abeb
Compare
|
🚧 Test build enqueued. |
|
🚧 Started test build. |
|
❌ Test build failed. Help
|
|
It looks like we may also have to change |
This PR updates Joplin to 3.4.12, which is the latest version at this time. It is in response to #283 to ensure that the version of Joplin packaged here is at the latest version.
Note: if preferred, I am willing to take over maintenance of it because I actively use Joplin for my school notes. At least, I can take over maintenance until an official developer decides to take it over. I say this because it appears that there might be a backlog of work, and since I use Joplin on a daily basis, I can assist with getting some of this work done.