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

if combining sphinx-last-updated-by-git with nbsphinx-link there is an obvious error occuring: The external linked file is not included in git and the path is therefore invalid #26

Closed
jannefleischer opened this issue Feb 16, 2021 · 2 comments · May be fixed by vidartf/nbsphinx-link#14

Comments

@jannefleischer
Copy link

Not really an issue for sphinx-last-updated-by-git because of the out-of-scope use via nbsphinx-link, nevertheless this may be important for some users and future reference.

sphinx-last-updated-by-git and nbsphinx-link are just not compatible with each other.

Traceback (most recent call last):
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\site-packages\sphinx\cmd\build.py", line 276, in build_main
    app.build(args.force_all, filenames)
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\site-packages\sphinx\application.py", line 349, in build
    self.builder.build_update()
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\site-packages\sphinx\builders\__init__.py", line 299, in build_update
    len(to_build))
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\site-packages\sphinx\builders\__init__.py", line 311, in build
    updated_docnames = set(self.read())
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\site-packages\sphinx\builders\__init__.py", line 424, in read
    for retval in self.events.emit('env-updated', self.env):
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\site-packages\sphinx\events.py", line 99, in emit
    results.append(callback(self.app, *args))
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\site-packages\sphinx_last_updated_by_git.py", line 166, in _env_updated
    update_file_dates(git_dir, dep_dates[git_dir])
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\site-packages\sphinx_last_updated_by_git.py", line 40, in update_file_dates
    stderr=subprocess.PIPE,
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\subprocess.py", line 356, in check_output
    **kwargs).stdout
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\subprocess.py", line 423, in run
    with Popen(*popenargs, **kwargs) as process:
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\subprocess.py", line 729, in __init__
    restore_signals, start_new_session)
  File "c:\users\fleischer\appdata\local\esri\conda\envs\monitoring\lib\subprocess.py", line 1017, in _execute_child
    startupinfo)
NotADirectoryError: [WinError 267] Der Verzeichnisname ist ungültig
@mgeier
Copy link
Owner

mgeier commented Feb 16, 2021

Thanks for the report!

This might be a bug in nbsphinx-link: vidartf/nbsphinx-link#14

Can you please try if that takes care of your error?

@mgeier
Copy link
Owner

mgeier commented Oct 5, 2023

My fix for nbsphinx-link has not (yet) been accepted, but there is nothing we can do here, so I'm closing this.

@mgeier mgeier closed this as not planned Won't fix, can't repro, duplicate, stale Oct 5, 2023
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 a pull request may close this issue.

2 participants