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

3.1.7: "paket install --log-file" blows up right away with IndexOutOfRangeException #1743

Closed
konste opened this Issue Jun 20, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@konste
Contributor

konste commented Jun 20, 2016

Documentation shows Filename parameter for --log-file command line switch as optional, but attempt to omit log file name causes Paket to break right away with IndexOutOfRangeException.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jun 21, 2016

Member

You mean if you don't provide a name but still set the switch?

Member

forki commented Jun 21, 2016

You mean if you don't provide a name but still set the switch?

@konste

This comment has been minimized.

Show comment
Hide comment
@konste

konste Jun 21, 2016

Contributor

Yes. From the doc it looks perfectly legal, although it does not tell what the default log file name is going to be.

Contributor

konste commented Jun 21, 2016

Yes. From the doc it looks perfectly legal, although it does not tell what the default log file name is going to be.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jun 21, 2016

Member

@eiriktsarpalis is this something Argu could handle?

Member

forki commented Jun 21, 2016

@eiriktsarpalis is this something Argu could handle?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jun 21, 2016

Member

sorry this is not coming from argu. fixing it now

Member

forki commented Jun 21, 2016

sorry this is not coming from argu. fixing it now

@forki forki closed this in 3621be1 Jun 22, 2016

forki added a commit that referenced this issue Jun 22, 2016

forki added a commit that referenced this issue Jun 22, 2016

eiriktsarpalis added a commit to eiriktsarpalis/Paket that referenced this issue Jun 30, 2016

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