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

RFC: fix #2501 #9554

Closed
wants to merge 2 commits into from
Closed

RFC: fix #2501 #9554

wants to merge 2 commits into from

Conversation

tkelman
Copy link
Contributor

@tkelman tkelman commented Jan 2, 2015

Build helpdb.jl instead of checking it in to repository.

In my local builds, there's a kinda funny race condition between the system image getting to base/help.jl vs the Sphinx build finishing, they happen at the same time with -j > 1. That could be fixed with a few more explicit makefile dependencies, but it's a little faster this way, and help.jl is near the end of the sysimg, well after inference which takes a good chunk of time. nevermind, not a race condition, helpdb.jl is a dependency of sys.o, not sys0.o. System image seems to build fine even without helpdb.jl present at all?

@tkelman
Copy link
Contributor Author

tkelman commented Jan 2, 2015

I don't know enough about Python virtualenvs to figure out what's happening on appveyor https://ci.appveyor.com/project/tkelman/julia/build/1.0.619/job/ypr5i09d4kqvkcs8:

New python executable in julia-env\Scripts\python.exe
Installing setuptools, pip...done.
/bin/sh: /c/projects/julia/deps/julia-env/bin/activate: No such file or directory
make[2]: *** [/c/projects/julia/deps/julia-env/bin/sphinx-build] Error 1

Any ideas?

@tkelman
Copy link
Contributor Author

tkelman commented Jan 2, 2015

Grr, Win32 Python you are awful and I hate you. Evidently activate and sphinx-build.exe end up in Scripts if you're using a Win32 Python, as opposed to bin for a posix Python. Anyone know the canonical way of determining whether or not a given Python is Win32?

@tkelman
Copy link
Contributor Author

tkelman commented Jan 2, 2015

@catawbasam thanks but I think 32 vs 64 bit shouldn't matter much as far as Sphinx is concerned. What I was referring to was whether the python executable is, say Cygwin-style, where conventions like /bin vs /Scripts match Unix systems, or Win32-style which is different because someone at some point felt /Scripts was a more Windows-y name than /bin.

edit: found when that was done, 4 days into virtualenv's history pypa/virtualenv@993ba13 "that seems to be the convention" grumble grumble

@tkelman tkelman changed the title WIP: fix #2501 RFC: fix #2501 Jan 3, 2015
@tkelman
Copy link
Contributor Author

tkelman commented Jan 3, 2015

Changing to RFC since I got it to work on AppVeyor. Should probably delete helpdb.jl, and maybe also $(build_docdir), in one of make clean or make cleanall. Thoughts?

@ViralBShah ViralBShah added the building Build system, or building Julia or its dependencies label Jan 8, 2015
@ViralBShah
Copy link
Member

I think make cleanall should be the target to delete $(build_docdir) in.

Does building helpdb.jl get triggered when any file in doc/stdlib it depends on changes? It doesn't seem to be the case right now, but I may be missing something.

@ViralBShah ViralBShah added the docs This change adds or pertains to documentation label Jan 8, 2015
@tkelman
Copy link
Contributor Author

tkelman commented Jan 8, 2015

Does building helpdb.jl get triggered when any file in doc/stdlib it depends on changes?

There is a dependency in doc/Makefile on stdlib/*.rst for the helpdb.jl target, so I would think so.

It doesn't seem to be the case right now, but I may be missing something.

Did you test this branch out and see it not rebuild when it should've? I admittedly should do more testing on this branch (and it needs a rebase) to get a feel for how well-behaved it is.

@tkelman
Copy link
Contributor Author

tkelman commented Jan 8, 2015

Rebased and added @rm -f doc/helpdb.jl $(build_docdir)/helpdb.jl to make cleanall. Let's see what CI thinks about it. I somehow managed to lose the Windows Python /Scripts fix in my first rebase, whoops. Dug it up on Travis and put it back though.

@tkelman
Copy link
Contributor Author

tkelman commented Jan 8, 2015

@jiahao any idea how to fix the unicode table warning? that would get pretty annoying to see in every CI log

@jiahao
Copy link
Member

jiahao commented Jan 8, 2015

The Unicode table simply needs to be rebuilt on master. Done in 27d6c1d

I added a new doc Make target to simplify rebuilding this table (it has to be done every time someone changes latex_symbols.jl). Now you can run make -C doc unicode.

tkelman added 2 commits April 18, 2015 00:15
mostly testing to see whether this saves a noticeable amount of time
add top-level Makefile rule to regenerate it

remove helpdb.jl in make cleanall

install-virtualenv in doc/helpdb.jl target
@tkelman
Copy link
Contributor Author

tkelman commented Jun 28, 2015

closing this since it looks like helpdb.jl will be going away imminently

@tkelman tkelman closed this Jun 28, 2015
@tkelman tkelman deleted the tk/fix2501 branch June 28, 2015 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
building Build system, or building Julia or its dependencies docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants