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

Bug: no goal for reports/doid.subclass.direct-in-mondo-only.tsv #402

Closed
wants to merge 2 commits into from

Conversation

joeflack4
Copy link
Contributor

@joeflack4 joeflack4 commented Jan 26, 2024

@joeflack4 joeflack4 requested a review from matentzn January 26, 2024 01:40
@joeflack4 joeflack4 marked this pull request as draft January 26, 2024 01:40
@joeflack4 joeflack4 self-assigned this Jan 26, 2024
@joeflack4 joeflack4 changed the title cleared reports dir Bug: no goal for reports/doid.subclass.direct-in-mondo-only.tsv Jan 26, 2024
@joeflack4 joeflack4 linked an issue Jan 26, 2024 that may be closed by this pull request
2 tasks
@joeflack4 joeflack4 added the bug Something isn't working label Jan 26, 2024
@joeflack4 joeflack4 mentioned this pull request Jan 26, 2024
#$(REPORTDIR)/%.subclass.direct-in-mondo-only.tsv:
#$(REPORTDIR)/%.subclass.added-obsolete.robot.tsv:

define sync_subclassof
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stuck at the moment

First, read how I got here: #397 (comment). Summary: I have a fresh repo now.

With fresh repo and using local make instead of ODK, now I'm getting this error message.

I conferred with my lord ChatGPT (chat) and it thinks my syntax for a multi target rule is correct. I mean, it was working before.

IMO I think the make error message is misleading.

I wonder if it might help to send an email to help-make@gnu.org.

I tried to resolve the error by making targets for each one of the targets in my multi-target rule, e.g. $(REPORTDIR)/%.subclass.direct-in-mondo-only.tsv. I have it call the code for what used to by my multi-target rule, which is now a function called sync_subclassof. That didn't work. However, if I make this goal:

reports/doid.subclass.direct-in-mondo-only.tsv: tmp/mondo-ingest.db tmp/mondo.db tmp/mondo.sssom.tsv

Now I no longer get the error: Bug: no goal for reports/doid.subclass.direct-in-mondo-only.tsv

Now the error I get is: make: *** No rule to make target tmp/mondo-ingest.db', needed by reports/doid.subclass.direct-in-mondo-only.tsv'. Stop.

If I comment out those dependencies tmp/mondo-ingest.db tmp/mondo.db tmp/mondo.sssom.tsv, then the goal will run... But obviously we need those.

Anywhere that's where I had to leave off for today.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dont worry about this, I will deal with this problem once I have my computer running. I would suggest you focus on ICD 11

@joeflack4 joeflack4 changed the title Bug: no goal for reports/doid.subclass.direct-in-mondo-only.tsv Bug: no goal for reports/doid.subclass.direct-in-mondo-only.tsv Jan 26, 2024
@joeflack4
Copy link
Contributor Author

@matentzn @twhetzel Closing this because this is to address #398, but that issue has been closed:
Nico wrote:

This has been fixed by #403

@joeflack4 joeflack4 closed this Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants