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.

Sign up for GitHub

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

Jump to bottom

CLI bug with paths in current directory in config #278

Closed
tarheel opened this issue Jun 9, 2019 · 2 comments
Closed

CLI bug with paths in current directory in config #278

tarheel opened this issue Jun 9, 2019 · 2 comments
Assignees
@moldover
Labels
bug certification-deliverable certification-deliverable-essential needs-clarification
Milestone
v1.0 Submission

Comments

@tarheel
Copy link
Contributor

tarheel commented Jun 9, 2019

If you run from the command line and specify a config file with no path (i.e. in the current dir), and that config files also lists a CVR source with no path, we end using the string "null" as the directory.
@tarheel tarheel added the bug label Jun 9, 2019
@tarheel tarheel added this to the v 1.0 Submission milestone Jun 9, 2019
@HEdingfield
Copy link
Contributor

Confirmed and noted this in #272:

I also discovered a minor weird CLI bug when running with Gradle rcv.bat (and possibly otherwise, and possibly only in Windows): directories specified for CVRs (and output) inject \null\ before... workaround requires using a path like "..\2013_minneapolis_mayor_cvr.xlsx" instead of "2013_minneapolis_mayor_cvr.xlsx"

@moldover moldover self-assigned this Jun 13, 2019
moldover added a commit that referenced this issue Jun 15, 2019
@moldover
fix for #278 use cwd with relative paths on the command line
59cfbe5
@moldover moldover mentioned this issue Jun 16, 2019
Merged
@moldover
Copy link
Contributor

Fixed with #287

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

@moldover moldover

Labels
bug certification-deliverable certification-deliverable-essential needs-clarification
Projects
None yet
Milestone
v1.0 Submission
Development

No branches or pull requests
6 participants
@tarheel @moldover @HEdingfield @chughes297 @gngilbert @freedomcounts