[mapguide-internals] Request for submission

[Robert Bray]

The only issue I see with that is that it will encourage discussion 
within Trac and lessens the visibility of patches (only those associated 
with the ticket will see it). This has a high potential for patches 
getting lost.

The first issue, discussion within the bug tracker is frowned upon by some.

As a solution, maybe the patch should be attached to the ticket and an 
e-mail should be sent to this list notifying developers of it's 
availability. Discussion of the patch could then take place on the list 
rather than in Trac.

Thoughts?

Bob



Andy Morsell wrote:
> I agree that the patch files themselves do not belong in the mailing list
> and should instead be in a centralized and organized place on the wiki 
> 
> 
> Andy 
> 
> -----Original Message-----
> From: mapguide-internals-bounces at lists.osgeo.org
> [mailto:mapguide-internals-bounces at lists.osgeo.org] On Behalf Of Jason Birch
> Sent: Friday, March 23, 2007 7:29 PM
> To: MapGuide Internals Mail List
> Subject: RE: [mapguide-internals] Request for submission
> 
> Folks,
>  
> I know that the instructions say that patches should go to the mailing list,
> but I'd really rather see these uploaded to the Trac Wiki attached to
> enhancement tickets.  This would allow for more centralised management of
> change requests.  Emails can then be sent to the list with a URL if desired.
>  
> Thoughts?
>  
> If not, we'll have to adjust the list to allow more attachments, something
> that I'm not really all that enthusiastic about :)
>  
> Jason
>  
> ________________________________
> 
> From: mapguide-internals-bounces at lists.osgeo.org on behalf of Rex Szeto
> (Intern)
> Subject: [mapguide-internals] Request for submission
> 
> Hi,
> 
> I would like to submit this patch to update couple files in viewerfiles
> folder for the Layerdefinition-1.0.0 to 1.1.0 change.
> 
> This update fixes problems with Buffer / Measure commands in the
> viewersample applications.
> 
> Note: apply patch inside OS\Web\src\viewerfiles
> 
> 
> 
> 
> _______________________________________________
> mapguide-internals mailing list
> mapguide-internals at lists.osgeo.org
> http://lists.osgeo.org/mailman/listinfo/mapguide-internals
>