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

reprun: command reprun_dataline is not recognized after running on main that changes the ado path #32

Open
Tracked by #16
luisesanmartin opened this issue Mar 14, 2024 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation reprun Relates to reprun
Milestone

Comments

@luisesanmartin
Copy link
Member

Tried reprun on a main do-file that changes the ado path with the line

sysdir set PLUS "${dos}/ado"

and it returned the error:

    Starting reprun. Creating the do-files for run 1 and run 2.
    Done creating the do-files for run 1 and run 2.
    Executing "00.Master.do" for run 1.
command reprun_dataline is unrecognized
r(199);

I think this is happening because the ado changes in the middle of the code execution of the input do-file to a path that doesn't have reprun or repkit installed and then doesn't recognize commands from reprun. Perhaps it would be better if it used the ado paths set by default when opening Stata for calling reprun commands instead of looking in the local ado paths of the do-file it's run on

@bbdaniels bbdaniels added the bug Something isn't working label Mar 28, 2024
@kbjarkefur kbjarkefur added the reprun Relates to reprun label Apr 16, 2024
@kbjarkefur
Copy link
Contributor

Hm, I am not sure it is possible to retrieve the default setting on that person's computer once it is changed with sysdir set PLUS "${dos}/ado". I wonder if the best solution here is to just install repkit in that PLUS folder. It can always be uninstalled afterwards

@kbjarkefur kbjarkefur removed their assignment Apr 16, 2024
@kbjarkefur kbjarkefur added documentation Improvements or additions to documentation and removed bug Something isn't working labels Apr 16, 2024
@kbjarkefur
Copy link
Contributor

Discussed with @bbdaniels, for now, just write in documentation that repkit needs to be installed in project ado if that is used

@bbdaniels
Copy link
Contributor

Duplicate of #3, see #33

@ankritisingh
Copy link
Contributor

Is this one done or should this be moved to next-release?

@ankritisingh ankritisingh modified the milestones: v2.2, next-release Jul 30, 2024
@ankritisingh ankritisingh modified the milestones: v3.0, next-release Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation reprun Relates to reprun
Projects
None yet
Development

No branches or pull requests

4 participants