Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
1 parent
596e827
commit 32a7ae7
Showing
1 changed file
with
107 additions
and
52 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
32a7ae7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems to cause
portindex
to fail on Apple Silicon Machines:32a7ae7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@barracuda156 Can you address this?
32a7ae7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will look for a similar case to borrow a fix. (These failures to parse happen quite often on PPC, so there should be some cases where that was fixed.)
32a7ae7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The way the portfile is written there is no version of this port that applies to "platform arm" -- only i386 and ppc are taken care of.
32a7ae7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kencu What is the correct way to deal with such cases?
32a7ae7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a block for arm, like the others... or some way of accounting for it. Add arm to another appropriate block perhaps. Or rejig the blocks to "ppc" and "not ppc"... etc.
it's a logic error, but not a hard one.
32a7ae7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, sure. I just thought maybe there is a standardized clause – there must be other ports like this.
I think I can do something like: if PPC > this, else > that.
32a7ae7
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
#17219