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

Allow ECO file parse to fail #10

Closed
wants to merge 1 commit into from

Conversation

ddugovic
Copy link
Contributor

@ddugovic ddugovic commented Jan 5, 2018

I don't have ../../eco.json.

@rpdelaney
Copy link
Collaborator

rpdelaney commented Jan 5, 2018

How did you install? It should be part of the package. You might have to initialize the submodule with something like git submodule init ./python-chess-annotator/annotator/eco I think

@ddugovic
Copy link
Contributor Author

ddugovic commented Jan 5, 2018

I installed via git clone https://github.com/rpdelaney/python-chess-annotator.git ; cd python-chess-annotator

but eco/eco.json is a symbolic link to a file ../../eco.json I don't have.

@rpdelaney
Copy link
Collaborator

Oh, I just looked again. I structured this weirdly, and I don't remember why. Try git submodule init ./python-chess-annotator/eco and that should help for now?

I think of the eco submodule as part of the app, so I have mixed feelings about skipping it if it's not found. That feels like masking a bug in installation. It seems to me I need to rethink how I've packaged this.

@ddugovic
Copy link
Contributor Author

ddugovic commented Jan 5, 2018

lila@dugovic-host:~/python-chess-annotator$ git submodule init ./python-chess-annotator/eco
error: pathspec './python-chess-annotator/eco' did not match any file(s) known to git.
lila@dugovic-host:~/python-chess-annotator$ git submodule init ./eco
Submodule 'eco' (git@github.com:niklasf/eco.git) registered for path 'eco'
lila@dugovic-host:~/python-chess-annotator$ cat eco/eco.json 
lila@dugovic-host:~/python-chess-annotator$ wc -l eco/eco.json 
0 eco/eco.json

@ddugovic
Copy link
Contributor Author

ddugovic commented Jan 5, 2018

OK, I found the file now:

lila@dugovic-host:~$ rm python-chess-annotator/ -rf
lila@dugovic-host:~$ git clone https://github.com/rpdelaney/python-chess-annotator.git ; cd python-chess-annotator
Cloning into 'python-chess-annotator'...
remote: Counting objects: 821, done.
remote: Compressing objects: 100% (39/39), done.
remote: Total 821 (delta 37), reused 71 (delta 37), pack-reused 745
Receiving objects: 100% (821/821), 147.56 KiB | 1.15 MiB/s, done.
Resolving deltas: 100% (480/480), done.
lila@dugovic-host:~/python-chess-annotator$ git submodule update
lila@dugovic-host:~/python-chess-annotator$ git submodule update --recursive
lila@dugovic-host:~/python-chess-annotator$ ls eco/
lila@dugovic-host:~/python-chess-annotator$ git submodule init
Submodule 'eco' (git@github.com:niklasf/eco.git) registered for path 'eco'
lila@dugovic-host:~/python-chess-annotator$ git submodule update --recursive
Cloning into '/home/lila/python-chess-annotator/eco'...
Enter passphrase for key '/home/lila/.ssh/id_rsa': 
Submodule path 'eco': checked out '8ef60f84461f885373bf4244ed908e7d8facb754'

@ddugovic ddugovic closed this Jan 5, 2018
@ddugovic ddugovic deleted the missing_eco branch January 5, 2018 17:44
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