Skip to content

fasm: init at 0.0.2 - #6

Merged
hansfbaier merged 1 commit into
openXC7:mainfrom
ngi-nix:moss/fasm
Oct 5, 2023
Merged

fasm: init at 0.0.2#6
hansfbaier merged 1 commit into
openXC7:mainfrom
ngi-nix:moss/fasm

Conversation

@jleightcap

@jleightcap jleightcap commented Sep 29, 2023

Copy link
Copy Markdown
Contributor

Add a fasm package.

Currently, the upstream ANTLR C++ runtime is not able to generate the necessary libraries for fasm to build against. So, this is falling back to the textx parser for now. I'll look into how this might be addressed in nixpkgs.

The tests are also broken upstream, which prevents some basic sanity checking.

Signed-off-by: Jack Leightcap <jack@leightcap.com>
@jleightcap

Copy link
Copy Markdown
Contributor Author

@hansfbaier do you happen to know the state of this package? it seems effectively unmaintained. i pulled in patches from this unmerged PR which fixed a lot of build system issues.

I might recommend forking the repo into openXC7 along with the build system improvements. Otherwise this is unlikely to be merged upstream into nixpkgs.

@hansfbaier

Copy link
Copy Markdown
Contributor

That's merged. Many thanks!

@albertchae albertchae mentioned this pull request Oct 8, 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.

2 participants