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

No ARM/macOS build for latest version #8

Closed
bevan-philip opened this issue Feb 25, 2023 · 2 comments
Closed

No ARM/macOS build for latest version #8

bevan-philip opened this issue Feb 25, 2023 · 2 comments

Comments

@bevan-philip
Copy link

bevan-philip commented Feb 25, 2023

Hi, cheers for this project!

I think I've ran into a variant of #5 and #6, where trying to install this on an ARM Mac machine only retrieves 0.1.0, as the latest build wasn't compiled for ARM (or at least, the CI workflow only targets x86_64).

It looks as if GitHub haven't got GitHub hosted runners for Mac/ARM (actions/runner#805 (comment)), so this might be too out of scope for at least the CI workflow.

Example of trying to force install 0.3.3:

(venv) bevan@Bevans-Air demo-parsing % pip install demoparser==0.3.3
ERROR: Could not find a version that satisfies the requirement demoparser==0.3.3 (from versions: 0.1.0)
ERROR: No matching distribution found for demoparser==0.3.3

(runtime error due to lack of feature as it's outdated

(venv) bevan@Bevans-Air demo-parsing % python dparser.py
Traceback (most recent call last):
  File "/Users/bevan/Documents/demo-parsing/dparser.py", line 5, in <module>
    events = parser.parse_events("round_end", props=["health"])
TypeError: DemoParser.parse_events() got an unexpected keyword argument 'props'

)

@LaihoE
Copy link
Owner

LaihoE commented Feb 26, 2023

Thanks for the issue. Yeah the 0.1.0 was probably me just testing around with the CI and not an actual release. I can revisit the CI after im done with a bigger feature for the parser. If you want to try building it from source then it should go something like this:

install before:
the rust compiler ie. https://www.rust-lang.org/tools/install
pip install maturin, patchelf

Then it should just be this:

git clone https://github.com/LaihoE/Python-demoparser.git
cd Python-demoparser/
maturin develop --release

If everything goes smoothly then it should install it to your current env.

@LaihoE
Copy link
Owner

LaihoE commented May 10, 2023

Looked at the progress they are having with the action and estimated was Q4/23 at the earliest. Yeah don't think this is happening before that (at least this s1 version).

@LaihoE LaihoE closed this as completed May 10, 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

No branches or pull requests

2 participants