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

Developer testing mode #10

Closed
zmughal opened this issue Mar 1, 2015 · 4 comments
Closed

Developer testing mode #10

zmughal opened this issue Mar 1, 2015 · 4 comments

Comments

@zmughal
Copy link
Member

zmughal commented Mar 1, 2015

Don't overwrite code gen files if they are the same so that make will not have to rebuild frequently. Simple sketch of the flow:

codegen $FILE.in > $FILE.new; md5sum $FILE $FILE.new; mv $FILE.new $FILE

Also, make sure all the files loaded during test are from the build directory rather than a pre-existing PDL install. Use strace to verify this.

@mohawk2
Copy link
Member

mohawk2 commented Mar 1, 2015

That's a good idea! I think checking %INC might be an easier way than strace.

@zmughal
Copy link
Member Author

zmughal commented Mar 5, 2015

OK, to clarify this more, if I have a .pd file that generates both native code and Perl code, I should not have to wait for things to compile if I only change the Perl part in places that don't change any of the compiled code. I'm using Perl for a reason!

Same for documentation.

@mohawk2
Copy link
Member

mohawk2 commented Mar 5, 2015

When I converted PDL::Ops (I think) to use Inline::Pdlpp, there's a PMCode param that triggers making a _*_int method name to be called by the included Perl. My initial look at PDL::PP didn't show how to trigger that without still leaving in the PMCode, even though that wasn't directly being used. I was then able to avoid recompiles where I only changed the Perl code / docs. What I'm saying is, this is a thing I have working.

@mohawk2
Copy link
Member

mohawk2 commented Dec 26, 2021

This is now implemented in the "multi-C" feature, making for extremely fast rebuilds.

@mohawk2 mohawk2 closed this as completed Dec 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants