Hi,

+1 binding

- vote good
- incubating in artefact name
- DISCLAIMER exists
- LICENCE and NOTICE good
- most file have Apache headers (see below)
- no binary files in release
- didn't test as don't have environment to test

These files do not have Apache headers:
  DeviceMap/App.config
  DeviceMap/DeviceMap.csproj
  DeviceMap/Properties/AssemblyInfo.cs
  DeviceMapConsole/App.config
  DeviceMapConsole/DeviceMapConsole.csproj
  DeviceMapConsole/Properties/AssemblyInfo.cs
  DeviceMapConsole/ua_strings.txt
  DeviceMap.sln

You may want to consider fixing for the next release:
- add apache headers to .config, .csproj, .cs, .sln and ua_strings.txt files
- year in NOTICE is wrong (should be 2014)
- host RC on Apache hardware when voting  - should be here 
https://dist.apache.org/repos/dist/dev/incubator/devicemap/

Thanks,
Justin

Reply via email to