[OpenLayers-Dev] Different lon and lat map
resolution (and other issues)
Jachym Cepicky
jachym.cepicky at gmail.com
Fri Nov 2 18:24:53 EDT 2007
Hi,
thank you for clarifying this. I add only small note, that when I'm
posting patch, than it is supposed to work for IE, Firefox and Opera and
normaly, it went through jslint as well.
Thanks again for your time for clarifying it
Jachym
Christopher Schmidt píše v Pá 02. 11. 2007 v 14:12 -0400:
> On Fri, Nov 02, 2007 at 06:37:43PM +0100, Jachym Cepicky wrote:
> > Hi,
> >
> > Tim Schaub p????e v P?? 02. 11. 2007 v 10:57 -0600:
> >
> > > > I can understand this. How many people do have write access to trunk?
> > > >
> > >
> > > http://trac.openlayers.org/wiki/CLA
> > >
> > > The above page describes who has access to what. As with all else in a
> > > wiki, this list requires a person to update it - so I can't say how
> > > current it is, but I think Erik probably is keeping it up.
> > >
> >
> > From what I see in mailing list, only few, who have the access are
> > really committing something or reviewing patches "from outside". Am I
> > wrong?
>
> Erik, Tim, Schuyler, and Eric have all actively reviewed patches in the
> past couple months. After pushing for weeks on end -- spending hours of
> my personal time every day -- I've been taking a break and sticking to
> only working 12 hours a day instead of 16. This leaves a lot less
> OpenLayers time there.
>
> I agree that there are fewer people reviewing and examining code than
> there could be. However, at the moment, I'm not aware of a solution
> which results in more people volunteering their free time to do
> integration of existing patches into the trunk. This isn't a case of
> avoiding code from outsiders, this is largely a case of avoiding any
> changes at all.
>
> I do have a goal of reviewing many of the outstanding patches, but at
> the moment, I can't afford to do it 'at work', so I need to do it in my
> free time. This means that anything complex doesn't get reviewed at all.
>
> Simple patches -- one or two lines -- especially those with tests, get a
> quick eyeball and review. Anything more complex is unlikely to proceed
> quickly, simply because it requires long solid blocks of time to move
> such complex patches forward.
>
> > > Will it work with the upcoming Proj4js work?
> >
> > No idea - I'm working with current subversion repository. Do I have to
> > apply all other proposed changes/patches in order to provide well tests?
>
> No, but it's important to be aware of upcoming work, in order to be able
> to discuss it with other members of the community with this in mind.
> It's not possible to know everything, but this particular change affects
> a pretty major part of the code -- right at the core -- and changing the
> core of OpenLayers behavior is something that I am personally adverse
> to.
>
> > If I'm posting a patch, than I can only say: "I tested everything, I
> > could. It works for me". When I find something, or somebody will report
> > something, that would not work, I will do my best to fix it.
> >
> > So, to answer your question: yes, everything works for me.
>
> Have you run the tests in Firefox and IE? That's typically a good
> question to have a ready answer to.
>
> > But the discussion went too concrete now. General question is: Do you
> > think, current system is all right? Are there enough reviewers, who are
> > checking the tracker regularly and communicating with the contributors?
> >
> > As you probably understand: I do not think so.
>
> I'll respond to this in a new thread, because I think it's a very
> important quesiton that deserves more than a piecemeal discussion as
> part of this thread.
>
> Regards,
--
Jachym Cepicky
e-mail: jachym.cepicky at gmail.com
URL: http://les-ejk.cz
GPG: http://www.les-ejk.cz/pgp/jachym_cepicky-gpg.pub
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Toto je =?UTF-8?Q?digit=C3=A1ln=C4=9B?=
=?ISO-8859-1?Q?_podepsan=E1?= =?UTF-8?Q?_=C4=8D=C3=A1st?=
=?ISO-8859-1?Q?_zpr=E1vy?=
Url : http://lists.osgeo.org/pipermail/openlayers-dev/attachments/20071102/101ee325/attachment.bin
More information about the Dev
mailing list