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

Create seil.cfg #290

Merged
merged 7 commits into from
Jul 3, 2014
Merged

Create seil.cfg #290

merged 7 commits into from
Jul 3, 2014

Conversation

kfinlay
Copy link
Contributor

@kfinlay kfinlay commented Jul 3, 2014

PCKeyboardHack was renamed Seil. The new version saves preferences to a renamed file.

PCKeyboardHack was renamed Seil. The new version saves preferences to a renamed file.
@lra
Copy link
Owner

lra commented Jul 3, 2014

Can you kill the old PCKeyboardHack application support, and merge the old synced files into this one?
Also can you add update the README and the changelog?
https://github.com/lra/mackup/blob/master/CONTRIBUTING.md

@kfinlay
Copy link
Contributor Author

kfinlay commented Jul 3, 2014

I get the impression that because the author open sourced PCKeyboardHack, a lot of people still run it under that name. I wasn't sure whether to combine them in a single config file. What do you recommend?

It raises the general question of how Mackup should deal with older versions of applications.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 47756b9 on kfinlay:patch-5 into b7404b6 on lra:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 47756b9 on kfinlay:patch-5 into b7404b6 on lra:master.

@lra
Copy link
Owner

lra commented Jul 3, 2014

Are the 2 still available? The 2 names are still valid? Any urls?

@kfinlay
Copy link
Contributor Author

kfinlay commented Jul 3, 2014

http://pqrs.org/macosx/keyremap4macbook/pckeyboardhack.html.en redirects to https://pqrs.org/macosx/keyremap4macbook/seil.html.en . The author recommends using the older version before OSX 10.8: https://github.com/tekezo/Seil . We could incorporate the PCKeyboardHack config data into seil.cfg since the name change is permanent. Again, this really calls for some sort of standard about archiving configurations from deprecated software. The name change possibility is probably pretty rare though.

@lra
Copy link
Owner

lra commented Jul 3, 2014

Yep, let's put PCKeyboardHack's files into Seil's conf.
Thanks!

It's configs have been moved to Seil, which is the new name of the software.
Pulled PCKeyboardBack's configs into this file.
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 032559e on kfinlay:patch-5 into b7404b6 on lra:master.

Removed PCKeyboardHack, since it's now named of Seil.
@kfinlay
Copy link
Contributor Author

kfinlay commented Jul 3, 2014

Ok, all done. Thank you.

lra added a commit that referenced this pull request Jul 3, 2014
@lra lra merged commit a95a072 into lra:master Jul 3, 2014
@kfinlay kfinlay deleted the patch-5 branch July 28, 2014 20:47
This pull request was closed.
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.

3 participants