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

add notes with scoop x64 #670

Closed
wants to merge 4 commits into from

Conversation

stokebreakup
Copy link

@stokebreakup stokebreakup commented May 8, 2024

No description provided.

x64 support json
@stokebreakup stokebreakup changed the title add scoop with notes.json add scoop May 8, 2024
@stokebreakup stokebreakup mentioned this pull request May 8, 2024
@stokebreakup stokebreakup changed the title add scoop add scoop x64 May 8, 2024
@stokebreakup stokebreakup changed the title add scoop x64 add notes with scoop x64 May 8, 2024
@stokebreakup stokebreakup marked this pull request as draft May 9, 2024 17:21
@stokebreakup stokebreakup reopened this May 10, 2024
@stokebreakup stokebreakup reopened this May 22, 2024
@guihkx
Copy link
Collaborator

guihkx commented May 22, 2024

Hey, thanks for working on this! :)

I haven't tested this properly yet, as I don't usually review draft pull requests, because they're technically not ready...

But anyway, you should probably update the notes.json file to use the Qt 6 build instead of the Qt 5 one, since the latter doesn't have the Kanban feature.

@guihkx
Copy link
Collaborator

guihkx commented May 22, 2024

I'm not familiar with how Scoop works, but after we install Notes using it, how do we open it?

I assume Notes can only be launched through the command line, and it won't be available in Windows' start menu... Or am I mistaken?

@stokebreakup
Copy link
Author

stokebreakup commented May 22, 2024

Hey, thanks for working on this! :)

Hi, thanks

I haven't tested this properly yet,

okay

as I don't usually review draft pull requests, because they're technically not ready...

I was wondering if you'd like to try it first, the scoop isn't very complicated.

> scoop install https://raw.githubusercontent.com/nuttyartist/notes/9781a347e38e2a5008f11281ea300850eb3967cd/notes.json
> scoop uninstall notes # "and or scoop cache rm *" remove all apps with issues, including notes

PS: Test on another machine or on a virtual machine before your personal machine.

But anyway, you should probably update the notes.json file to use the Qt 6 build instead of the Qt 5 one, since the latter doesn't have the Kanban feature.

There is a way to auto-update, but I still don't understand how it works or how to do it correctly with scoop.

I'm not familiar with how Scoop works, but after we install Notes using it, how do we open it?

Once you download the file and unzip it, it will look like a command line. I didn't make a complete script that creates a shortcut effectively.

scoop notes  # open notes.exe

Once you download the file and unzip it, it will look like a command line.

yes. exactly, command line.

, and it won't be available in Windows' start menu...

I didn't make a complete script that creates a shortcut effectively.

Or am I mistaken?

you are sure. And that's why I open a draft here. I was wondering where I should store notes.json, or if I should create notesx86.json, notesx64.json or something more complete. Or how this script should work. If I get more information, this pull request goes from being a draft to something more complete.

@guihkx
Copy link
Collaborator

guihkx commented May 22, 2024

Thanks for clarifying things!

In my opinion, since Notes is obviously a graphical-only program, having a shortcut in the start menu to open it is a must. If this is indeed doable with Scoop, I'd prefer having that before we could merge it.

I was wondering where I should store notes.json, or if I should create notesx86.json

I'm not sure what other contributors think, but in my opinion, having a single notes.json file only for the x64 Qt 6 build is preferable.

We pretty much only offer the x86 Qt 5 build because, apparently, some of our users are still on Windows 7, where Qt 6 isn't officially supported.

@stokebreakup
Copy link
Author

stokebreakup commented May 22, 2024

In my opinion, since Notes is obviously a graphical-only program, having a shortcut in the start menu to open it is a must. If this is indeed doable with Scoop, I'd prefer having that before we could merge it.

thanks for info.

I'm not sure what other contributors think

For this other reason, I open a draft pull request. So I'll wait for new information from other contributors, I'd rather send something that everyone agrees on

but in my opinion, having a single notes.json file only for the x64 Qt 6 build is preferable.

yes.

I was wondering where I should store notes.json,

I was asking if I should submit the pull request to the scoop community or here

@guihkx
Copy link
Collaborator

guihkx commented May 22, 2024

I was asking if I should submit the pull request to the scoop community or here

I definitely think it would make more sense to submit to the Scoop community instead.

Based on a quick look I took at how Scoop works, they seem to have their own official repositories, so installing Notes from one of those would be much better, IMO.

However, if they have a way to "import" the Scoop file from third-party repositories (i.e., from here into their own repositories), I think it would be even better.

Just so we can "control" the release process better, directly from here.

@stokebreakup
Copy link
Author

update: ScoopInstaller/Extras#13419

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants