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

remove auto-generated files #19

Merged
merged 1 commit into from
Jan 25, 2017

Conversation

wchristian
Copy link
Collaborator

As discussed in IRC, the *.inc files should not be in the repo in the first places. This commit removes them, puts them in .gitignore and ensures the files are generated when MfPL is being run, thus making it easy to cut releases with the proper files.

This obsoletes PR #12.

@devel-chm
Copy link
Collaborator

Thanks for the fix. I'll make the pull once I figure out how to resolve the conflicts with the current git head.

@wchristian
Copy link
Collaborator Author

I rebased it onto master, only needed the newline fixes applied for git to be able to make sense of it.

Note in general: Don't use warn for debug output. Output on stdout, warnings and errors on stderr.

@devel-chm devel-chm merged commit 7581e1b into Perl-GPU:master Jan 25, 2017
@devel-chm
Copy link
Collaborator

Thanks. I'm working on making generate-XS.pl generate the needed files starting with the priority list used by prima, microidium, app-shadertoy, and pdl. See the branch chm_generate-XS_work for details including a directory pogl_implemented with lists of the routines implemented in OpenGL and their variations with suffixes.

@wchristian wchristian deleted the no_auto_generated_files branch January 25, 2017 20:55
@wchristian
Copy link
Collaborator Author

chm_generate-XS_work looks like you merged after a rebase, which is superfluous and messes up the structure.

image

You'd wanna reset the branch to 1cab718 and do a force push instead.

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.

None yet

2 participants