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

signed net35, net40 and net45 #46

Merged
merged 2 commits into from Sep 16, 2013
Merged

Conversation

prabirshrestha
Copy link
Contributor

signed desktop profiles

This PR will fail as NLua.dll tries to reference non-signed assemblies.

You will need to merge these NLua/KeraLua#9 and NLua/KopiLua#12 first and update the git submodules so it references the signed assemblies before you merge this in.

@viniciusjarina
Copy link
Member

Hi @prabirshrestha

This is awesome 😄

Can you check why Travis is complaining about your PR? https://travis-ci.org/NLua/NLua/builds/11389481#L341

Thank you

@prabirshrestha
Copy link
Contributor Author

signed libraries can only reference signed libraries. The reason it is failing is coz the signed NLua is trying to reference unsigned KeraLupa and KopiLua.

Once you merge the signed KeraLupa and KopiLupa PR I sent, I will update the git submodule so it references the signed kopilua and keralua. That would then fix the Travis complains.

@prabirshrestha
Copy link
Contributor Author

updated the git submodules. it is green now.

viniciusjarina added a commit that referenced this pull request Sep 16, 2013
signed net35, net40 and net45
@viniciusjarina viniciusjarina merged commit b7d0b94 into NLua:master Sep 16, 2013
@prabirshrestha prabirshrestha deleted the signed branch September 16, 2013 00:19
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