You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@crail.apache.org by Jonas Pfefferle <pe...@japf.ch> on 2018/10/24 08:04:23 UTC

[CANCEL][VOTE] Release of Apache Crail-1.1-incubating [rc2]

All,


I'm canceling the vote for Apache Crail 1.1-incubating release candidate 2, 
due to found issues (see below).
I will prepare a new release candidate.

Regards,
Jonas

  On Wed, 24 Oct 2018 09:26:34 +0200
  "Jonas Pfefferle" <pe...@japf.ch> wrote:
> Hi Julian,
> 
> 
> Comments inline.
> 
> Thanks,
> Jonas
> 
>  On Tue, 23 Oct 2018 12:33:41 -0700
>  Julian Hyde <jh...@apache.org> wrote:
>> -1 (binding) due to missing build instructions and included binary 
>>files
>>
>> Other reviewers please take note: this is the first release to 
>>include binary artifacts. 1.0 only included a src.tar.gz, whereas 
>>this also has a bin.tar.gz. I have no clue how to review binary 
>>artifacts. Should they even be included in a release vote?
> 
> We did not come up with this. I've checked multiple other Apache 
>projects that do it like this.
> 
>> 
>> Downloaded, checked signatures, LICENSE, NOTICE, DISCLAIMER; 
>>successfully built on Linux JDK 8 using “mvn -DskipTests install”; 
>>successfully ran RAT.
>> 
>> Checked that contents of the src.tar.gz match git at the same 
>>commit:
>> 
>> $ diff -r . ~/open1/crail/
>> Only in .: DEPENDENCIES
>> Only in ./doc: build
>> Only in ./doc/source: _static
>> Only in /home/jhyde/open1/crail/: .git
>> 
>> The doc/build directory is a concern, as it contains binary .doctree 
>>files. Remove it?
> 
> Yes, that is an over site. The git repository does not include the 
>build directory and _static as it should be.
> I will remove it in the src tarball and create a new rc.
> 
>> 
>> Related to the previous point, you’ve excluded doc/ from RAT 
>>checking. Why? If RAT were enabled, it would have noticed the binary 
>>files.
> 
> I don't think it is feasible to put license headers in each rst 
>file. I checked multiple other projects
> and non of them do this, that is why I excluded the doc directory. I 
>will modify the rule to only exclude rst files.
> 
>> 
>> The directory prefix in src.tar.gz is crail-parent-1.1-incubating. 
>>It would be better if it matched the release name, i.e. 
>>apache-crail-1.1-incubating-src. It did in 1.0-rc3, so I’m not sure 
>>why we’ve regressed.
> 
> We switched to the Apache parent pom for the release and have not 
>figured out yet how to fix the naming and some other issues, but we 
>have opened a JIRA to investigate: 
>https://jira.apache.org/jira/projects/CRAIL/issues/CRAIL-56
> 
> 
>> There are no build instructions. I gave you a pass in the last 
>>release[1] but we need them now.
> 
> Our new documentation (linked in the readme and on the webpage) 
>talks about how to build Crail:
> https://incubator-crail.readthedocs.io/en/latest/source.html
> Or do you want this to be part of the call for vote email?
> 
>> 
>> Julian
>> 
>> [1] 
>>https://lists.apache.org/thread.html/7c3dc2c6ea1d2ac2550b7b6937d4f29965f0eb418bb92e296556df9e@%3Cdev.crail.apache.org%3E 
>><https://lists.apache.org/thread.html/7c3dc2c6ea1d2ac2550b7b6937d4f29965f0eb418bb92e296556df9e@%3Cdev.crail.apache.org%3E> 
>>> On Oct 23, 2018, at 8:00 AM, bernard metzler <bm...@gmx.ch> 
>>>wrote:
>>> 
>>> Hi Jonas,
>>> 
>>> Thanks a lot for preparing that release candidate!
>>> 
>>> I built from source and installed and test-ran Crail
>>> both from built source and binary download. Looks
>>> all pretty good to me!
>>> 
>>> +1 from my side
>>> 
>>> Cheers,
>>> Bernard.
>>> 
>>> 
>>> On 10/22/2018 15:27, Jonas Pfefferle wrote:
>>>> Hi all,
>>>> This is a call for a vote on releasing Apache Crail 1.1-incubating, 
>>>>release candidate rc2.
>>>> The source and binary tarball, including signatures, digests, etc. 
>>>>can be found at:
>>>> https://dist.apache.org/repos/dist/dev/incubator/crail/1.1-rc2/
>>>> The commit to be voted upon:
>>>> https://git-wip-us.apache.org/repos/asf?p=incubator-crail.git;a=commit;h=2b75c4c47e2491c860079e1be985b6767f226061
>>>> The Nexus Staging URL:
>>>> https://repository.apache.org/content/repositories/orgapachecrail-1001/
>>>> Release artifacts are signed with the key AA557B11:
>>>> https://www.apache.org/dist/incubator/crail/KEYS
>>>> For information about the contents of this release, see:
>>>> https://git-wip-us.apache.org/repos/asf?p=incubator-crail.git;a=blob;f=HISTORY.md;h=86e1b5a39d3a13f1cca2621677057763fdf9ac96;hb=2b75c4c47e2491c860079e1be985b6767f226061 
>>>>or for better readability 
>>>>https://github.com/apache/incubator-crail/blob/v1.1-rc2/HISTORY.md
>>>> Please vote on releasing this package as Apache Crail 1.1-incubating
>>>> The vote will be open for 72 hours.
>>>> [ ] +1 Release this package as Apache Crail 1.1-incubating
>>>> [ ] +0 no opinion
>>>> [ ] -1 Do not release this package because ...
>>>> Thanks,
>>>> Jonas
>>> 
>>