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

Error with Package Control installation #44

Closed
sentience opened this Issue Apr 22, 2013 · 7 comments

Comments

Projects
None yet
5 participants
@sentience

Installing Flatland using Package control produces this error:

Error loading colour scheme Packages/flatland/Theme - Flatland/Widget - Flatland.stTheme: Error parsing plist xml: Failed to open file in file "Packages/flatland/Theme - Flatland/Widget - Flatland.stTheme"

This is due to Flatland having been written assuming it would be installed in a package directory named flatland. Package Control installs it in a directory named "Theme - Flatland".

@mwagstaff

This comment has been minimized.

Show comment
Hide comment
@mwagstaff

mwagstaff Apr 22, 2013

Yep, got hit by that, too. The workaround is to go into your Packages directory and rename the "Theme - Flatland" directory you see there to "flatland".

Yep, got hit by that, too. The workaround is to go into your Packages directory and rename the "Theme - Flatland" directory you see there to "flatland".

@unimatrixZxero

This comment has been minimized.

Show comment
Hide comment
@unimatrixZxero

unimatrixZxero Apr 22, 2013

I went into my package directory and created a symlink named flatland that points to the them directory. Worked out fine for me.

cd ~/Library/Application\ Support/Sublime\ Text\ 2/Packages
ln -s Theme\ -\ Flatland flatland 

I went into my package directory and created a symlink named flatland that points to the them directory. Worked out fine for me.

cd ~/Library/Application\ Support/Sublime\ Text\ 2/Packages
ln -s Theme\ -\ Flatland flatland 
@joshnh

This comment has been minimized.

Show comment
Hide comment
@joshnh

joshnh Apr 23, 2013

I can't get the theme to work through package control no matter what I try.

joshnh commented Apr 23, 2013

I can't get the theme to work through package control no matter what I try.

@kevva

This comment has been minimized.

Show comment
Hide comment
@kevva

kevva Apr 23, 2013

Contributor

@sentience, what's the best solution here? To patch this repo or change name in repositories.json from Theme - Flatland to flatland?

Contributor

kevva commented Apr 23, 2013

@sentience, what's the best solution here? To patch this repo or change name in repositories.json from Theme - Flatland to flatland?

@sentience

This comment has been minimized.

Show comment
Hide comment
@sentience

sentience Apr 23, 2013

@kevva Renaming the package to flatland in repositories.json would be a quick fix, but would prevent many people from finding your package when searching Package Control for themes.

I recommend instead you update Flatland to assume its package directory will be named Theme - Flatland. It appears that’s what you’ve done with pull request #46. Of course, this will cause the opposite problem, where users who perform a manual installation of Flatland by cloning the repo into their Packages directory will have the wrong directory name. It appears you’ve done your best to update README.md with the appropriate instructions, however.

@kevva Renaming the package to flatland in repositories.json would be a quick fix, but would prevent many people from finding your package when searching Package Control for themes.

I recommend instead you update Flatland to assume its package directory will be named Theme - Flatland. It appears that’s what you’ve done with pull request #46. Of course, this will cause the opposite problem, where users who perform a manual installation of Flatland by cloning the repo into their Packages directory will have the wrong directory name. It appears you’ve done your best to update README.md with the appropriate instructions, however.

@sentience

This comment has been minimized.

Show comment
Hide comment
@sentience

sentience Apr 23, 2013

Note how the Soda Theme’s manual installation instructions include a Git clone command that sets the installation directory name to "Theme - Soda" to deal with this same issue.

Note how the Soda Theme’s manual installation instructions include a Git clone command that sets the installation directory name to "Theme - Soda" to deal with this same issue.

@kevva

This comment has been minimized.

Show comment
Hide comment
@kevva

kevva Apr 23, 2013

Contributor

@sentience, yeah, updated PR with Git install instructions.

Contributor

kevva commented Apr 23, 2013

@sentience, yeah, updated PR with Git install instructions.

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