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

Fix Nemerle compilation (Do same as .Net) #242

Merged
merged 1 commit into from Mar 11, 2012
Merged

Fix Nemerle compilation (Do same as .Net) #242

merged 1 commit into from Mar 11, 2012

Conversation

NN---
Copy link
Contributor

@NN--- NN--- commented Mar 11, 2012

The original discussion is here:
https://bugzilla.novell.com/show_bug.cgi?id=689533
And this is the proposed patch:
https://bugzillafiles.novell.org/attachment.cgi?id=426501

Please accept this pull request it is essential to enable self compilation of Nemerle (https://github.com/rsdn/nemerle) in Linux.

@Sectoid
Copy link
Contributor

Sectoid commented Mar 11, 2012

+1 to this pull request.

In the bugzilla someone mentioned this patch causes regression in smcs, but:

  1. i haven't found any info regarding what exactly it breaks;
  2. moonlight seems to be dead anyway ;)

So again, +1, plz accept this.

@konrad-kruczynski
Copy link
Contributor

+1 for Nemerle on Linux (with packages etc) too.

vargaz added a commit that referenced this pull request Mar 11, 2012
Fix Nemerle compilation (Do same as .Net)
@vargaz vargaz merged commit 8814ddb into mono:master Mar 11, 2012
@vargaz
Copy link
Contributor

vargaz commented Mar 11, 2012

mono master no longer uses reflection.emit for c# compilation, so this can be merged now. If it breaks something, at least it won't be the C# compiler.

@Sectoid
Copy link
Contributor

Sectoid commented Mar 11, 2012

Also, please merge this to mono-2-10 branch.

@vargaz
Copy link
Contributor

vargaz commented Mar 11, 2012

The 2.10 branch uses the old reflection.emit based compilers, so it will probably break something there.

@Sectoid
Copy link
Contributor

Sectoid commented Mar 11, 2012

How can i prove that patch doesn't break anything? I ask bcs i see no problems in my build with the patch. But of course i can miss something.

@NN---
Copy link
Contributor Author

NN--- commented Mar 11, 2012

Agree with Sectoid ? What are the tests we must run to make sure there are no problems ?

@vargaz
Copy link
Contributor

vargaz commented Mar 11, 2012

merged it to 2.10.

@ghost
Copy link

ghost commented Mar 19, 2012

Great work.

@viniciusjarina viniciusjarina mentioned this pull request May 30, 2012
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Fix Nemerle compilation (Do same as .Net)

Commit migrated from mono/mono@8814ddb
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

4 participants