# \[patch\] agdlr: Fix for Package.NormalizePath

**URL:** <https://www.ruby-forum.com/t/patch-agdlr-fix-for-package-normalizepath/164457>\
**Category:** IronRuby\
**Created:** [April 1, 2009, 5:01pm UTC](https://www.ruby-forum.com/t/patch-agdlr-fix-for-package-normalizepath/164457 "2009-04-01T17:01:54Z")\
**Posts on this page:** 5\
**Page:** 1

<div class="post-metadata">

**Author:** ![Jb\_Evain](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/67e7ee/32.png) [@Jb\_Evain](https://www.ruby-forum.com/u/Jb_Evain)\
**Post date:** [April 1, 2009, 5:01pm UTC](https://www.ruby-forum.com/t/patch-agdlr-fix-for-package-normalizepath/164457/1 "2009-04-01T17:01:54Z")

</div>

Hey,

This patch is part of a bunch I sent to Jimmy privately. But they  
haven’t been applied, so am resending them here for review.

Package.NormalizePath transforms paths in the xap that use \ as a  
directory separator to /. So use \ directly, instead of expecting it  
to be the system directory separator.

Thanks,

---

<div class="post-metadata">

**Author:** ![Jb\_Evain](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/67e7ee/32.png) [@Jb\_Evain](https://www.ruby-forum.com/u/Jb_Evain)\
**Post date:** [April 1, 2009, 7:38pm UTC](https://www.ruby-forum.com/t/patch-agdlr-fix-for-package-normalizepath/164457/2 "2009-04-01T19:38:10Z")

</div>

They are applied in the dev/mono branch, and I can apply them for 0.5.1.

~Jimmy  
Sent from my phone

---

<div class="post-metadata">

**Author:** ![Jb\_Evain](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/67e7ee/32.png) [@Jb\_Evain](https://www.ruby-forum.com/u/Jb_Evain)\
**Post date:** [April 1, 2009, 8:04pm UTC](https://www.ruby-forum.com/t/patch-agdlr-fix-for-package-normalizepath/164457/3 "2009-04-01T20:04:41Z")

</div>

Hey Jimmy,

On 4/1/09, Jimmy S. [removed\_email\_address@domain.invalid](mailto:removed_email_address@domain.invalid) wrote:

> They are applied in the dev/mono branch, and I can apply them for 0.5.1.

Yeah, that would be cool. Is there any reason for not merging them in  
origin?

---

<div class="post-metadata">

**Author:** ![Jb\_Evain](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/67e7ee/32.png) [@Jb\_Evain](https://www.ruby-forum.com/u/Jb_Evain)\
**Post date:** [April 1, 2009, 8:14pm UTC](https://www.ruby-forum.com/t/patch-agdlr-fix-for-package-normalizepath/164457/4 "2009-04-01T20:14:41Z")

</div>

No major reason … they are small enough to be considered bug reports,  
so I have no problem making the fixes. FYI, I have to be more careful  
with substantial improvements/features, for the same reason that  
IronRuby.dll or the DLR can’t accept contributions, but I hope to have a  
clearer message on contributions in the future.

I don’t have moonlight on any of my machines to test it out (though I  
trust you =P). Can you point me at instructions for setting up  
Moonlight? I have a Ubuntu VM , or is there another distro I should use  
since you’re testing more on it (SUSE I assume?).

~js

---

<div class="post-metadata">

**Author:** ![Jb\_Evain](https://www.ruby-forum.com/letter_avatar_proxy/v4/letter/j/67e7ee/32.png) [@Jb\_Evain](https://www.ruby-forum.com/u/Jb_Evain)\
**Post date:** [April 1, 2009, 9:06pm UTC](https://www.ruby-forum.com/t/patch-agdlr-fix-for-package-normalizepath/164457/5 "2009-04-01T21:06:21Z")

</div>

On 4/1/09, Jimmy S. [removed\_email\_address@domain.invalid](mailto:removed_email_address@domain.invalid) wrote:

> No major reason … they are small enough to be considered bug reports, so I have no problem making the fixes. FYI, I have to be more careful with substantial improvements/features, for the same reason that IronRuby.dll or the DLR can’t accept contributions, but I hope to have a clearer message on contributions in the future.

Cool. I don’t particularly care that my own personal patch is applied,  
I care about having the feature working. So if you guys prefer bug  
reports I can submit bugs reports.

> I don’t have moonlight on any of my machines to test it out (though I trust you =P). Can you point me at instructions for setting up Moonlight? I have a Ubuntu VM , or is there another distro I should use since you’re testing more on it (SUSE I assume?).

I use openSUSE, but there’s no reason you couldn’t make it work  
Ubuntu. You would have to setup Mono from svn, I suggest using:

[http://www.mono-project.com/Parallel\_Mono\_Environments](http://www.mono-project.com/Parallel_Mono_Environments)

There’s some instructions for Moonlight here:

[http://www.mono-project.com/Moonlight#Developing\_Moonlight](http://www.mono-project.com/Moonlight#Developing_Moonlight)

But it’s not always up to date as we are currently changing things a  
lot. Don’t hesitate to ask on moonlight-list for help, or contact me  
directly.

[http://lists.ximian.com/mailman/listinfo/moonlight-list](http://lists.ximian.com/mailman/listinfo/moonlight-list)
