Agree with Sergey - the source archive should be build independently, perhaps
as a part of the release target.

Fixing source manually won't work, because then the content of the release
tarball will be different from the release commit. These seem to be a
relatively point-fixes and should require the full release re-testing, IMO.

Thanks
  Cos

On Sat, Dec 26, 2015 at 07:20PM, Anton Vinogradov wrote:
> Hi,
> 
> 1-2)
> Seems that we have to have dotnet build folder located at "regular" java
> target folder which will be automatically excluded at source package build.
> Another way is to configure "source-release-assembly" at "apache-release"
> profile somehow.
> And last one case is to build source package at separate step (without
> using dotnet assemblies)
> 
> 1 & 2 way is preffered.
> 
> Thoughts?
> 
> 3) We have tip about this issue at
> https://apacheignite.readme.io/docs/getting-started, but I agree with
> Vladimir.
> 
> Cos, in case we will fix sourcess manually (for issue 1-2) and regenerate
> md5 & sha will you change your decision?
> 
> On Sat, Dec 26, 2015 at 2:29 PM, Sergey Kozlov <skoz...@gridgain.com> wrote:
> 
> > Hi
> >
> > I analyzed the build log for Apache Ignite 1.5.0.final-rc1 and found that
> > we make a source archive after building of fabric edition. It seems it
> > caused the issue raised by Cos.
> > I propose to move the step of source archiving  before any build steps and
> > may be run RAT against source zip file.
> >
> >
> >
> > On Sat, Dec 26, 2015 at 11:12 AM, Vladimir Ozerov <voze...@gridgain.com>
> > wrote:
> >
> >> Cos,
> >>
> >> p.1 and p.2 appear to be the same - folder "modules/platforms/dotnet/bin"
> >> should not exist in source release, only in binary build.
> >> p.3 - looks like we need make .NET and CPP docs creation step off by
> >> default and enable it only if particular flag is set. For now behavior is
> >> the opposite - docs are generated by default unless "!isClientDocs" flag
> >> is
> >> set.
> >>
> >>
> >> On Sat, Dec 26, 2015 at 10:46 AM, Konstantin Boudnik <c...@apache.org>
> >> wrote:
> >>
> >> > Looks like a -1 [binding] is order
> >> >
> >> > 1) RAT check fails on
> >> >     modules/platforms/dotnet/bin/Apache.Ignite.Core.xml
> >> > 2) release archive includes binary files under
> >> >     /modules/platforms/dotnet/bin
> >> >     /modules/platforms/dotnet/bin/x86/
> >> > 3) build fails with
> >> > [ERROR] Failed to execute goal
> >> > org.apache.maven.plugins:maven-antrun-plugin:1.7:run (dotnetdoc) on
> >> project
> >> > ignite-clients: An Ant BuildException has occured: Execute failed:
> >> > java.io.IOException: Cannot run program "doxygen" (in directory
> >> >
> >> >
> >> "/ws/build/ignite-hadoop/rpm/BUILD/ignite-hadoop-1.5.0.final/modules/platforms/dotnet"):
> >> > error=2, No such file or directory
> >> >
> >> > Cheers,
> >> >   Cos
> >> >
> >> > On Fri, Dec 25, 2015 at 04:00PM, Anton Vinogradov wrote:
> >> > > Dear Sirs!
> >> > >
> >> > > We have uploaded release candidate to
> >> > > https://dist.apache.org/repos/dist/dev/ignite/1.5.0.final-rc1/
> >> > >
> >> > > This is very important release containing initial versions of
> >> Ignite.NET
> >> > > and Ignite C++ and many other important features.
> >> > >
> >> > > Tag name is
> >> > > 1.5.0.final-rc1
> >> > >
> >> > > 1.5.0 changes:
> >> > > * Ignite.NET: Initial Release.
> >> > > * Ignite C++: Initial Release.
> >> > > * Massive performance improvements for cache operations and SQL.
> >> > > * Added new binary cache object marshalling implementation.
> >> > > * Added IgniteSemaphore data structure.
> >> > > * Added MQTT Streamer.
> >> > > * Added Twitter Streamer.
> >> > > * Added Ignite Sink (integration with Apache Flume).
> >> > > * Fixed optimistic serializable transactions: implemented
> >> "deadlock-free"
> >> > > locking algorithm.
> >> > > * Fixed failover for continuous queries.
> >> > > * Fixed compilation and runtime errors under OpenJDK and IBM JDK.
> >> > > * Fixed Integer.size limitation for cache.
> >> > > * Fixed and improved cache types configuration.
> >> > > * Fixed cache rebalancing.
> >> > > * Many stability and fault-tolerance fixes.
> >> > > * Added example to demonstrate the usage of EntryProcessor.
> >> > >
> >> > > Complete list of closed issues:
> >> > >
> >> >
> >> https://issues.apache.org/jira/issues/?jql=project%20%3D%20IGNITE%20AND%20fixVersion%20%3D%201.5%20AND%20status%20%3D%20closed
> >> > >
> >> > > DEVNOTES
> >> > >
> >> >
> >> https://git-wip-us.apache.org/repos/asf?p=ignite.git;a=blob_plain;f=DEVNOTES.txt;hb=refs/tags/1.5.0.final-rc1
> >> > >
> >> > > RELEASENOTES
> >> > >
> >> >
> >> https://git-wip-us.apache.org/repos/asf?p=ignite.git;a=blob_plain;f=RELEASE_NOTES.txt;hb=refs/tags/1.5.0.final-rc1
> >> > >
> >> > > Please start voting.
> >> > >
> >> > > +1 - to accept Apache Ignite 1.5.0.final
> >> > > 0 - don't care either way
> >> > > -1 - DO NOT accept Apache Ignite 1.5.0.final (explain why)
> >> > >
> >> > > This vote will go for 72 hours.
> >> >
> >>
> >
> >
> >
> > --
> > Sergey Kozlov
> >
> >

Attachment: signature.asc
Description: Digital signature

Reply via email to