[mapguide-internals] please review the patch for Ticket #1153

Jason Birch jason at jasonbirch.com
Tue Nov 17 02:23:06 EST 2009


There are likely some reasons (ahem, lgpl) why this is included in
mgos. IMO, it should be a configurable item rather than hard-coded
though. I think there's a ticket for this...

On 2009-11-16, Buddy Hu <Buddy.Hu at autodesk.com> wrote:
> Yes, this is a MGE path. In my opinion, this code snippet is useless for
> MGOS. I know it's a little weird to put these code here. But it has existed
> years. I think we should remove it from MGOS later.
> Basically, for fix this ticket, this patch is ok.
>
> Bruce & Tom, can you give some suggestion for this? Is there any side-effect
> to MGOS?
>
> Thanks,
> -Buddy
>
> -----Original Message-----
> From: mapguide-internals-bounces at lists.osgeo.org
> [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Zac Spitzer
> Sent: Tuesday, November 17, 2009 9:54 AM
> To: MapGuide Internals Mail List
> Subject: Re: [mapguide-internals] please review the patch for Ticket #1153
>
> but thats a MGE specific path?
>
> On Tue, Nov 17, 2009 at 12:42 PM, Buddy Hu <Buddy.Hu at autodesk.com> wrote:
>
>> Hi All,
>>
>> I have attached a patch for ticket #1455 for your review. This patch
>> updates the default path to the Mentor dictionaries to use on the Linux,
>> when the MENTOR_DICTIONARY_PATH is not defined in the system.
>>
>>
>> Ticket URL: <https://trac.osgeo.org/mapguide/ticket/1153>
>>
>>
>>
>> If It's ok, please submit it for me. Because I don't have the right to
>> submit code to MapGuide open source.
>>
>> Thanks,
>> -Buddy Hu
>>
>> _______________________________________________
>> mapguide-internals mailing list
>> mapguide-internals at lists.osgeo.org
>> http://lists.osgeo.org/mailman/listinfo/mapguide-internals
>>
>
>
>
> --
> Zac Spitzer
> Solution Architect / Director
> Ennoble Consultancy Australia
> http://www.ennoble.com.au
> http://zacster.blogspot.com
> +61 405 847 168
> _______________________________________________
> mapguide-internals mailing list
> mapguide-internals at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/mapguide-internals
> _______________________________________________
> mapguide-internals mailing list
> mapguide-internals at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/mapguide-internals
>


More information about the mapguide-internals mailing list