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

Possible alternative method for Inline::Module #27

Closed
mohawk2 opened this issue Dec 22, 2014 · 3 comments
Closed

Possible alternative method for Inline::Module #27

mohawk2 opened this issue Dec 22, 2014 · 3 comments

Comments

@mohawk2
Copy link
Contributor

mohawk2 commented Dec 22, 2014

Currently, I::M makes a blib/Inline, then works from there.

The alternative idea is that instead of working into blib/Inline and trying to hide your work, it could generate a *.xs file next to the *.pm, in the same spirit as a *.xs is used to make a *.c, then *.o, etc. Then the normal build system could take over and install the *.so into blib, as normal.

Likely benefits are this will reduce the amount of code in hooking into the build systems eg EUMM.

@ingydotnet
Copy link
Owner

I::M doesn't make a blib/Inline, it just tells Inline to use that as its DIRECTORY.

The whole point of Inline::Module is to just provide some minimal glue that let's Inline.pm and Inline::C work like normal and then have the results be in a place where module frameworks need them at various times. Inline::Module is currently under 500 lines and the plugins for dzil and module-build and such are tiny.

However I like the gist of your idea. I reinterpret it to mean, do more stuff author side and dist it, if the same thing would always happen user side. We might be able to get to just XS files but I'd rather take a safer refactoring approach.

@mohawk2
Copy link
Contributor Author

mohawk2 commented Dec 22, 2014

It would be interesting if there were a method to make Inline::C return its *.xs file, probably akin to bind. That would make this task almost trivial.

@mohawk2
Copy link
Contributor Author

mohawk2 commented Jan 3, 2015

Looks like @sisyphus had a similar idea: https://github.com/sisyphus/inlinex-c2xs

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