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

few update #146

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

few update #146

wants to merge 12 commits into from

Conversation

sobuj53
Copy link

@sobuj53 sobuj53 commented Apr 4, 2021

Sqlite db update and few other update to make Clipjump working

icon updated
updated to change folder 📂 path `\` to `/` for using in R
Sqlite dll update to latest version
Updated to solve issue aviaryan#134
@Skrell
Copy link

Skrell commented Apr 4, 2021

Thank you so much for keeping the best clipboard manager on the internet working!!! <3 How do I generate a new Clipjump.exe to use with your source code?

not needed
@Skrell
Copy link

Skrell commented Apr 4, 2021

When I try running your forked code with the original Clipjump.exe I get an error saying it can't find the sqlite3.dll which I can clearly see is present?

As Clipjump asks for Run as Admin, that's why added Runas Admin method to ease the workload on user.
For some reason Exiting Clipjump from tray doesn't properly close the process. That's why added Nircmd force close method to completely close on exit. But this requires Nircmd to be installed in, `%USERPROFILE%\AppData\Local\Microsoft\WindowsApps`. Just download Nircmd from https://www.nirsoft.net/utils/nircmd-x64.zip and extract to the mentioned folder.
@sobuj53
Copy link
Author

sobuj53 commented Apr 4, 2021

Hi thank you for your interest. You're welcome to fork my repo or download the content as zip. Clipjump.exe won't work in this case because I didn't create executable file with the latest changes so it'll throw out error. Just use Clipjump_x64.ahk, it'll do the job. You can create a task schedule to automate this process. If you want to create task schedule then do select Run with highest privileges in general tab, set triggers at Log on at actions tab start a program, Program/Script value = "C:\Program Files\AutoHotkey\AutoHotkey.exe" arguments value = "<path>\Clipjump_x64.ahk" of course change <path> to your Clipjump directory.

@Skrell
Copy link

Skrell commented Apr 4, 2021

Would you mind making a new Clipjump.exe ?

@sobuj53
Copy link
Author

sobuj53 commented Apr 4, 2021

Would you mind making a new Clipjump.exe ?

nope, as it's basically the same thing and also it gives me freedom to change or update the code without hassle.

Added Release update and Nircmd Dependency explanation.
@Skrell
Copy link

Skrell commented Apr 4, 2021

Ok I did what you suggested and it appeared to work fine, except for 2 things:

  1. in Task explorer the process shows up as "Autohotkey Unicode 64-bit" instead of "Clipjump"
  2. When I attempt to exit clipjump from the tray menu I get the error "Nircmd not found! Download Nircmdm from, https://www.nirsoft.net/utils/nircmd.html" ??

@sobuj53
Copy link
Author

sobuj53 commented Apr 4, 2021

  1. I can't do anything about that, it’s by design.
  2. Nircmd is required for this build read, https://github.com/sobuj53/Clipjump

@Skrell
Copy link

Skrell commented Apr 4, 2021

Why is nircmd required and why does it have to be in that directory? Can you change your scripts to at least use nircmd from the directory I run clipjump.ahk in?

@sobuj53
Copy link
Author

sobuj53 commented Apr 5, 2021

I specifically wrote why it’s required in the following commit sobuj53@a74b303
In short it’s not possible or maybe possible, you have source code so you can always modify it to your need. I saw it fits my requirement so I adopted it.

Removed External Nircmd dependency and replaced with simple function.
Removed Nircmd dependency and replaced with simple ahk built in function.
Updated SQLite.dll to 3.35.4
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.

None yet

2 participants