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

loadFiles subdirectory bug #237

Merged
merged 2 commits into from
Sep 5, 2015
Merged

loadFiles subdirectory bug #237

merged 2 commits into from
Sep 5, 2015

Conversation

goehle
Copy link
Member

@goehle goehle commented Sep 3, 2015

This was an extra line left over from some testing which prevented loadModules from using paths. Test by trying to load setOrientation problem 14. Before the pull you should get a missing macro warning and after you shouldn't.

@mgage
Copy link
Member

mgage commented Sep 5, 2015

These are ok. There needs to be some changes to localOverrides.conf which I'll submit as part of the ltiups3 pull request.

$pg{specialPGEnvironmentVars}{CAPA_Tools} = "$courseDirs{templates}/Contrib/CAPA/macros/CAPA_Tools/",
$pg{specialPGEnvironmentVars}{CAPA_MCTools} = "$courseDirs{templates}/Contrib/CAPA/macros/CAPA_MCTools/",

(before the paths were written in full and read_whole_problem_file won't handle those. )
This also requires a link Contrib in the templates directory of each course.

mgage added a commit that referenced this pull request Sep 5, 2015
loadFiles subdirectory bug
@mgage mgage merged commit 586a071 into openwebwork:develop Sep 5, 2015
@goehle goehle deleted the subdir branch July 15, 2016 15:17
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 this pull request may close these issues.

2 participants