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

doc2 outputs in current work dir #6583

Closed
dom96 opened this issue Oct 24, 2017 · 3 comments · Fixed by #14324
Closed

doc2 outputs in current work dir #6583

dom96 opened this issue Oct 24, 2017 · 3 comments · Fixed by #14324
Labels
Documentation Content Related to documentation content (not generation). Path Handling Tools

Comments

@dom96
Copy link
Contributor

dom96 commented Oct 24, 2017

This isn't consistent with the way output binaries are generated. (This is possibly a duplicate).

@dom96 dom96 added Documentation Content Related to documentation content (not generation). Tools labels Oct 24, 2017
@FedericoCeratto
Copy link
Member

Confusingly, "nim doc" behaves like "nim c".

@narimiran
Copy link
Member

Tested with v0.19.0:

Both nim doc and nim doc2 create outputs in the subdirectory. E.g. nim doc lib/pure/collections/sequtils.nim produces ./lib/pure/collections/sequtils.html.

@timotheecour
Copy link
Member

timotheecour commented May 8, 2020

this should be re-opened; a regression happened since 0.19.0 and it outputs again in $pwd.
see getOutFile (result = conf.outDir / changeFileExt(filename, ext) which resolves to $pwd/changeFileExt(filename, ext) when conf.outDir is empty)

the bug is that "" / path resolves to path instead of giving an error

@dom96 dom96 reopened this May 9, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 12, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 12, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 13, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 13, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 20, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 20, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 21, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 21, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 23, 2020
timotheecour added a commit to timotheecour/Nim that referenced this issue May 23, 2020
Araq pushed a commit that referenced this issue May 25, 2020
…any bug fixes with nim doc (#14324)

* refs #6583 fix nim doc output
* changelog
* change default for outDir when unspecified
* cleanups
* --project implies --index
EchoPouet pushed a commit to EchoPouet/Nim that referenced this issue Jun 13, 2020
… all projects, many bug fixes with nim doc (nim-lang#14324)

* refs nim-lang#6583 fix nim doc output
* changelog
* change default for outDir when unspecified
* cleanups
* --project implies --index
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Content Related to documentation content (not generation). Path Handling Tools
Projects
None yet
4 participants