Re: [Geotools-devel] Hudson geotools-release lists wildly inaccurate Changes

[Ben Caradoc-Davies]

Ah, thanks, Justin. That explains everything.

On 15/05/13 22:56, Justin Deoliveira wrote:
> Yeah, the hudson job isn't really set up for this. The first thing
> hudson does us update master to grab the latest scripts, which is where
> you are seeing that change log from.

-- 
Ben Caradoc-Davies 
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

--
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] Hudson geotools-release lists wildly inaccurate Changes

[Justin Deoliveira]

Yeah, the hudson job isn't really set up for this. The first thing hudson
does us update master to grab the latest scripts, which is where you are
seeing that change log from. And then the next thing it does is checkout
the branch to release from, as specified by the script parameter.


On Wed, May 15, 2013 at 2:43 AM, Ben Caradoc-Davies <
ben.caradoc-dav...@csiro.au> wrote:

> So, in releasing 9.2, I saw the change list, and thought that writing the
> release notes was going to be straightforward:
> http://hudson.opengeo.org/**hudson/view/All/job/geotools-**release/60/
>
> And then I noticed bleeding-edge changes that I was certain should not be
> on stable:
> http://hudson.opengeo.org/**hudson/view/All/job/geotools-**
> release/60/changes#detail8
> Commit bd684ec5e0fdb1d7b41c10d8e4cd10**e1cf9fd58d by Niels Charlier
> GEOS-5779 Implementation of SortBy in App-Schema
>
> Manual inspection of git log on 9.x confirms that this change is *not* on
> 9.x. Manual inspection of the source code confirms that this change is not
> in the 9.2 source bundle.
>
> So, is this a known problem? Is this just Hudson scraping the console log
> and getting the wrong idea? Perhaps we need a big notice in the release
> guide warning release engineers to ignore Hudson when writing release notes.
>
> Kind regards,
>
> --
> Ben Caradoc-Davies 
> Software Engineer
> CSIRO Earth Science and Resource Engineering
> Australian Resources Research Centre
>



-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.
--
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] Hudson geotools-release lists wildly inaccurate Changes

[Ben Caradoc-Davies]

On 15/05/13 16:50, Andrea Aime wrote:
> On Wed, May 15, 2013 at 10:47 AM, Andrea Aime
> mailto:andrea.a...@geo-solutions.it>> wrote:
> We always used jira to write the release notes (in other words, if a
> commit does not have
> an associated ticket, closed in time, it's as if it never occurred).
> Never had a look at Hudson's output honestly, and it seems it is not
> setup to provide a changelog
> anyways.
> +1 to warn the release manager not to look at it
>
> Btw, once the release is published one way to get the real list of
> changes should be:
>
> git fetch -t geotools (or whatever your remote for the official repo is
> called)
> git log 9.1..9.2
>
> Right now you cannot run it because 9.2 does not exist yet on the
> official git repo (the
> publish script creates it).

Thanks. It took me a little while to realise that I needed fetch -t to 
get tags not reachable from branch heads. I should note this and your 
git-log advice in the guide as well.

Local clean build went fine. All artifacts look good. Publish is 
currently stuck in the Hudson traffic jam, but shouldn't be long now. 
Release notes not needed until Monday.

Kind regards,

-- 
Ben Caradoc-Davies 
Software Engineer
CSIRO Earth Science and Resource Engineering
Australian Resources Research Centre

--
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d
___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] Hudson geotools-release lists wildly inaccurate Changes

[Andrea Aime]

On Wed, May 15, 2013 at 10:47 AM, Andrea Aime
wrote:

> We always used jira to write the release notes (in other words, if a
> commit does not have
> an associated ticket, closed in time, it's as if it never occurred).
> Never had a look at Hudson's output honestly, and it seems it is not setup
> to provide a changelog
> anyways.
> +1 to warn the release manager not to look at it
>

Btw, once the release is published one way to get the real list of changes
should be:

git fetch -t geotools (or whatever your remote for the official repo is
called)
git log 9.1..9.2

Right now you cannot run it because 9.2 does not exist yet on the official
git repo (the
publish script creates it).

Cheers
Andrea

-- 
==
GeoServer training in Milan, 6th & 7th June 2013!  Visit
http://geoserver.geo-solutions.it for more information.
==

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054  Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39  339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

---
--
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Re: [Geotools-devel] Hudson geotools-release lists wildly inaccurate Changes

[Andrea Aime]

On Wed, May 15, 2013 at 10:43 AM, Ben Caradoc-Davies <
ben.caradoc-dav...@csiro.au> wrote:

> So, in releasing 9.2, I saw the change list, and thought that writing
> the release notes was going to be straightforward:
> http://hudson.opengeo.org/hudson/view/All/job/geotools-release/60/
>
> And then I noticed bleeding-edge changes that I was certain should not
> be on stable:
>
> http://hudson.opengeo.org/hudson/view/All/job/geotools-release/60/changes#detail8
> Commit bd684ec5e0fdb1d7b41c10d8e4cd10e1cf9fd58d by Niels Charlier
> GEOS-5779 Implementation of SortBy in App-Schema
>
> Manual inspection of git log on 9.x confirms that this change is *not*
> on 9.x. Manual inspection of the source code confirms that this change
> is not in the 9.2 source bundle.
>
> So, is this a known problem? Is this just Hudson scraping the console
> log and getting the wrong idea? Perhaps we need a big notice in the
> release guide warning release engineers to ignore Hudson when writing
> release notes.
>

We always used jira to write the release notes (in other words, if a commit
does not have
an associated ticket, closed in time, it's as if it never occurred).
Never had a look at Hudson's output honestly, and it seems it is not setup
to provide a changelog
anyways.
+1 to warn the release manager not to look at it

Cheers
Andrea


-- 
==
GeoServer training in Milan, 6th & 7th June 2013!  Visit
http://geoserver.geo-solutions.it for more information.
==

Ing. Andrea Aime
@geowolf
Technical Lead

GeoSolutions S.A.S.
Via Poggio alle Viti 1187
55054  Massarosa (LU)
Italy
phone: +39 0584 962313
fax: +39 0584 1660272
mob: +39  339 8844549

http://www.geo-solutions.it
http://twitter.com/geosolutions_it

---
--
AlienVault Unified Security Management (USM) platform delivers complete
security visibility with the essential security capabilities. Easily and
efficiently configure, manage, and operate all of your security controls
from a single console and one unified framework. Download a free trial.
http://p.sf.net/sfu/alienvault_d2d___
GeoTools-Devel mailing list
GeoTools-Devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel