Skip to content

Require SWIG#21

Open
pgorczak wants to merge 1 commit into
BastilleResearch:masterfrom
pgorczak:patch-1
Open

Require SWIG#21
pgorczak wants to merge 1 commit into
BastilleResearch:masterfrom
pgorczak:patch-1

Conversation

@pgorczak

@pgorczak pgorczak commented Aug 2, 2017

Copy link
Copy Markdown

No description provided.

@matt-knight

Copy link
Copy Markdown
Contributor

Thanks @pgorczak! Checking to see whether I need to get a CLA from you before I can merge -- will circle back once I have an answer.

@matt-knight matt-knight added the bug label Aug 3, 2017
@matt-knight

Copy link
Copy Markdown
Contributor

Hi Philipp, do you work for an organization or company? I have to ask because you may have signed an IP agreement that asserts ownership to your work during your employment. In such case, I may need a corporate CLA to ensure there are no conflicts there.

You're the first contributor to this project so I'm still getting this formalized. Thanks for your help and patience!

@pgorczak

pgorczak commented Aug 8, 2017

Copy link
Copy Markdown
Author

I do work for a university and I think a CLA would put us on the safe side. If it makes things easier, I'll close this PR so you can make the change yourself 😄

@matt-knight

Copy link
Copy Markdown
Contributor

Hi Philipp,

I apologize for the delay -- I thought I had sent our CLA off to you, but I was mistaken! Would you please fill out our CLA and email it back to me? I'll be happy to accept your PR once the lawyers are happy :)

Link to the document: https://github.com/BastilleResearch/Bastille_CLA

Thanks again for your contribution!

Best,
Matt

@sajanikapukotuwa

Copy link
Copy Markdown

wow these things are complex :-)

@E3V3A

E3V3A commented Dec 31, 2017

Copy link
Copy Markdown

Seriously!?

@sneak

sneak commented Nov 25, 2019

Copy link
Copy Markdown

It's GPL. Why would you need a CLA unless you require the rights assignment so that you can sell it to someone else under a different license?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants