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

ams-common: remove "sources" line from A-line.conf #25

Closed
wants to merge 1 commit into from

Conversation

targetdisk
Copy link
Contributor

The sources line causes build.pl to look for the nonexistent 68k/ams/common//common directory. Tested to still work with A-line under MacRelix after the fix.

The sources line causes build.pl to look for the nonexistent
68k/ams/common//common directory.  Tested to still work with A-line
under MacRelix after the fix.
@jjuran
Copy link
Owner

jjuran commented Jun 20, 2023

Thanks! I'm not sure how that got in there or persisted for so long, but glad to see it gone.

However, that change makes build.pl ams-common fail much more noisily, which I fixed by adding a platform directive: a9a44d4

@targetdisk
Copy link
Contributor Author

Thanks! I'm not sure how that got in there or persisted for so long, but glad to see it gone.

However, that change makes build.pl ams-common fail much more noisily, which I fixed by adding a platform directive: a9a44d4

Looks good to me!

@jjuran
Copy link
Owner

jjuran commented Jun 20, 2023

Okay, pushed to master.

@jjuran jjuran closed this Jun 20, 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 this pull request may close these issues.

None yet

2 participants