Re: [VOTE] Release Apache Traffic Control 1.8.0-incubating (RC8)

2017-01-20 Thread Justin Mclean
Hi,

> Thanks again for your time checking our release, and sorry we’re still not 
> there. 

It’s a complex release and may take a little while to get right, and it ’s 
certainly not expected that you get everything right while in the incubator.

In this case the issues where raised with the last release and not fixed. Do 
you have these issues recorded in JIRA? That a good way to remember they need 
to be done before the next release.

Also you still may get enough other people to voting +1 on this release (a -1 
is not a veto) or you or someone else could change my mind.

> We’ve been running https://creadur.apache.org/rat/ 
> 

Rat is a very useful tool, but it wont catch everything and only knows about a 
limited number of licenses.

>  - is there anything else that you use that we can also use? Any tips you 
> have for us to get through this?

Just follow [1] it gives clear instruction off how to deal with most cases. [2] 
is also a help when it come to allowed 3rd party code. Your mentors should also 
be able to help here.

If you want me to take a look at when it comes up on your dev list, before 
offering it up for voting on the incubator list, just ask.

Thanks,
Justin

1. http://www.apache.org/dev/licensing-howto.html
2. https://www.apache.org/legal/resolved
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



Re: [VOTE] Release Apache Traffic Control 1.8.0-incubating (RC8)

2017-01-20 Thread Jan van Doorn
Hi Justin, 

Thanks again for your time checking our release, and sorry we’re still not 
there. 

We’ve been running https://creadur.apache.org/rat/ 
  - is there anything else that you use that 
we can also use? Any tips you have for us to get through this?

Best Regards,
JvD


> On Jan 20, 2017, at 4:20 PM, Justin Mclean  wrote:
> 
> Hi,
> 
> Sorry but it’s -1 (binding) as licensing issues and file headers reported 
> last release that were not fixed.
> 
> I checked:
> - names include incubating
> - signature and hashes are OK
> - disclaimer exits
> - LICENSE still has issues brought up from last release
> - NOTICE is OK but has wrong year
> - Source files have ASF header. A couple of files have them incorrectly.
> - No unexpected binary files in release.
> 
> While some of the issues brought up in last release have been fixed, there a 
> large number that look like they haven’t.
> 
> LICENSE Is still missing licenses for jQuery UI, normalize.css, angular 
> loading bar, lz-string (which also incorrectly has an ASF header on it), 
> pretty print  (and which also has an incorrectly added ASF header) and 
> modernizr. They may be others. Also the dual licensing for select2.js has not 
> been dealt with and nor has the licensing of multiple font files.
> 
> LICENSE also includes WebAppers Progress Bar which as pointed out last 
> release is Category B and cannot be included in a source release.
> 
> Re licensing around the MaxMind DB GeoLite2 Database [1] (under a CC 
> share-alike license) also IMO needs to be sorted. I’m not sure it classifies 
> as "unmodified media”, best to ask on legal discuss.
> 
> Thanks,
> Justin
> 
> 1. ./traffic_router/core/src/test/resources/geo/GeoLite2-City.mmdb.gz
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
> 



Re: [VOTE] Release Apache Traffic Control 1.8.0-incubating (RC8)

2017-01-20 Thread Justin Mclean
Hi,

Sorry but it’s -1 (binding) as licensing issues and file headers reported last 
release that were not fixed.

I checked:
- names include incubating
- signature and hashes are OK
- disclaimer exits
- LICENSE still has issues brought up from last release
- NOTICE is OK but has wrong year
- Source files have ASF header. A couple of files have them incorrectly.
- No unexpected binary files in release.

While some of the issues brought up in last release have been fixed, there a 
large number that look like they haven’t.

LICENSE Is still missing licenses for jQuery UI, normalize.css, angular loading 
bar, lz-string (which also incorrectly has an ASF header on it), pretty print  
(and which also has an incorrectly added ASF header) and modernizr. They may be 
others. Also the dual licensing for select2.js has not been dealt with and nor 
has the licensing of multiple font files.

LICENSE also includes WebAppers Progress Bar which as pointed out last release 
is Category B and cannot be included in a source release.

Re licensing around the MaxMind DB GeoLite2 Database [1] (under a CC 
share-alike license) also IMO needs to be sorted. I’m not sure it classifies as 
"unmodified media”, best to ask on legal discuss.

Thanks,
Justin

1. ./traffic_router/core/src/test/resources/geo/GeoLite2-City.mmdb.gz
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org



[VOTE] Release Apache Traffic Control 1.8.0-incubating (RC8)

2017-01-20 Thread Dan Kirkwood
Hello Incubator PMC,

The Apache Traffic Control community has voted on and approved a
proposal to release Apache Traffic Control 1.8.0-incubating.  We now
kindly request that the Incubator PMC members review and vote on this
incubator release.

The VOTE RESULT is here:

https://lists.apache.org/thread.html/64b9ffad20aca8a4af3bdb0ce33c2d2a26c24c48d96c31494acdca9d@%3Cdev.trafficcontrol.apache.org%3E

The draft release notes (along with links to artifacts,
signatures/checksums, and updated documentation) can be found here:

http://trafficcontrol.incubator.apache.org/downloads/

The git tag for the repository is "RELEASE-1.8.0-RC8":
https://github.com/apache/incubator-trafficcontrol/releases/tag/RELEASE-1.8.0-RC8

The source distribution (also linked in the release notes) is here:

https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/1.8.0/RC8/

Build instructions are included in the BUILD.md file which is
included in the source artifact.

Artifacts have been signed with the "dang...@apache.org" key listed in:

https://dist.apache.org/repos/dist/dev/incubator/trafficcontrol/KEYS

Please review and vote:

[  ] +1 Approve the release
[  ] -1 Don't approve the release (please provide specific comments)

This vote will be open for at least 72 hours.

Thanks,

- Dan

-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org