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

Documentation update list #33

Closed
smeech opened this issue Jan 2, 2024 · 9 comments
Closed

Documentation update list #33

smeech opened this issue Jan 2, 2024 · 9 comments

Comments

@smeech
Copy link
Collaborator

smeech commented Jan 2, 2024

I put a list of documentation updates on the main repo Issue list, but it might fit better here.

@traptw1thin
Copy link

Another update I would propose is for https://espanso.org/docs/sync/. One, I don't think that Google Drive works for Linux due to the file system not showing the names of files but only their ids. But two, and I'm not sure if this is only on Linux (and Dropbox) but I think it is erroneous to say that you want to move the $CONFIG folder into the newly created espanso folder within the place you want to sync from. More accurate would be to say to move the contents of the $CONFIG folder into this newly created folder. Then delete the $CONFIG folder and then modify the command (at least for linux) to ln -s "/home/user/Dropbox/espanso" "/home/user/.config" basically replacing the $CONFIG folder. Otherwise when you restart espanso you end up with the file paths "/home/user/.config/espanso/" and "/home/user/.config/espanso/espanso/" and inside the first one when you restart espanso you get another match and config folder resulting in another set of base.yml and everything else that would be very confusing if not non-functional if there were contradictions. Additionally there should be a mention that you need to run espanso stop before starting this or else very odd things will happen when you move things out of the $CONFIG folder.

@smeech
Copy link
Collaborator Author

smeech commented Jan 29, 2024

That makes sense.
To shorten the instructions, I think you are saying:

  • Determine $CONFIG result of espanso path config
  • espanso stop
  • Create a new "espanso" folder in the cloud storage
  • Move the contents of the $CONFIG folder into it
  • Link command "new espanso folder" to $CONFIG
  • espanso start

Is that correct?

The principle difference is changing the word "in" to "to" at the end of the first sentence in each section? Then it's up to the individual to decide whether to create a new folder and move the contents into it, or to move the folder itself.

@traptw1thin
Copy link

traptw1thin commented Jan 29, 2024

That MIGHT work, I don't know because thats not quite what I did or meant, but it may be equivalent. I will explain the slight difference in case it does make a difference.

The second to last bullet you listed is not quite what I was trying to describe. The $CONFIG that you get in the first bullet from espanso path config will be something like /home/user/.config/espanso however that second to last bullet would need to be:

  • Link command "new espanso folder" to /home/user/.config (in Linux obviously diff for diff OS, just trying to express the difference)

The difference is per what you wrote above the second to last bullet's command would expand $CONFIG to /home/user/.config/espanso. Again, possibly what you wrote would also work but I think it would result in the double espanso/espanso directory structure still.

Otherwise your summary of what I was saying perfectly summarizes my long explanation

@traptw1thin
Copy link

Additionally in just attempting to get this sync setup on a Windows PC I also discovered the order of commands is backwards from what is on that page. In windows with mklink the order is <link location> <target> but that page of the documentation lists it as <target> <link location>

@smeech
Copy link
Collaborator Author

smeech commented Jan 29, 2024

Would you refresh your view of docs, please, and recheck? I did make a change in the Windows mklink order recently, so I think it should now be correct.

@smeech
Copy link
Collaborator Author

smeech commented Jan 29, 2024

The difference is per what you wrote above the second to last bullet's command would expand $CONFIG to /home/user/.config/espanso. Again, possibly what you wrote would also work but I think it would result in the double espanso/espanso directory structure still.

I'm not sure if this is correct. I tried the process here to test it out, and ran the command:
ln -s "/home/stephen/Cloud/Dropbox/espanso" /home/stephen/.config/espanso
which is what the docs suggest, and it worked as I expected.

@traptw1thin
Copy link

traptw1thin commented Feb 1, 2024

When I do ln -s "/home/charity/Dropbox/espanso" "/home/charity/.config/espanso" I end up with the link located at /home/user/.config/espanso/espanso, which I don't think is correct. This results in the following folder structure:
image
But if I delete the espanso folder at /home/user/.config and then run ln -s "/home/charity/Dropbox/espanso" "/home/charity/.config" I get the following folder structure, which I thought was the goal:
image

Maybe its a misunderstand on my part, but with the first folder structure there ends up being 2 default.yml files, 2 packages folders, and 2 base.yml files. One set at /home/user/.config/espanso/config/default.yml, /home/user/.config/espanso/match/packages/, and /home/user/.config/espanso/match/base.yml and the other set at /home/user/.config/espanso/espanso/config/default.yml, /home/user/.config/espanso/espanso/match/packages/, and /home/user/.config/espanso/espanso/match/base.yml. If the intent is to have two sets then maybe that's fine.

@smeech
Copy link
Collaborator Author

smeech commented Feb 1, 2024

There should only be one set of configuration files.
I think the trick is perhaps to ensure the /espanso folder doesn't exist in /home/user/.config before running the link command? This was mentioned in the macOS instructions, but not the others.
I have put a warning above all three in my docs repo for consideration.

@smeech
Copy link
Collaborator Author

smeech commented Feb 13, 2024

New issue started #38 for further conversation.

@smeech smeech closed this as completed Feb 13, 2024
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

No branches or pull requests

2 participants