PS it was my call to make a custom assembly for this repo. What happened before was that you could barely see the IDL files due to clutter with the dummy files needed to make a maven distribution. I would have really been confused to see this stuff, especially if I were a non java person. As the repo is for multi-language and it is an IDL project with no compilation here or otherwise, I made the judgement call to do a custom assembly.
In other words, I'll help revert it if that's not acceptable. However, I think it will be detrimental to the usefulness of the distribution and possibly another user would as well. On Fri, Apr 26, 2019 at 8:48 AM Adrian Cole <[email protected]> wrote: > > Hi, Zoltan. > > The api is an IDL project only, there is nothing to build. The actual > source here is all protos, swagger and thrift IDLs. > > What's probably distracting is that we currently have a partial source > distribution in maven to depend on the protos (though we only use them > in tests). > If you ignore the ceremony files, there's only one: zipkin.proto The > zip created by this build is actually the official and complete > source dist. > > $ jar -tvf ./target/zipkin-proto3-0.2.2-SNAPSHOT.jar > 460 Fri Apr 26 07:40:28 MYT 2019 META-INF/MANIFEST.MF > 0 Fri Apr 26 07:40:28 MYT 2019 META-INF/ > 0 Fri Apr 26 07:40:28 MYT 2019 META-INF/maven/ > 0 Fri Apr 26 07:40:28 MYT 2019 META-INF/maven/org.apache.zipkin.proto3/ > 0 Fri Apr 26 07:40:28 MYT 2019 > META-INF/maven/org.apache.zipkin.proto3/zipkin-proto3/ > 10147 Fri Apr 26 07:40:28 MYT 2019 zipkin.proto > 263 Fri Apr 26 07:40:28 MYT 2019 META-INF/DEPENDENCIES > 170 Fri Apr 26 07:40:28 MYT 2019 META-INF/NOTICE > 9379 Fri Apr 26 07:35:26 MYT 2019 > META-INF/maven/org.apache.zipkin.proto3/zipkin-proto3/pom.xml > 11358 Fri Apr 26 07:40:28 MYT 2019 META-INF/LICENSE > 112 Fri Apr 26 07:40:28 MYT 2019 > META-INF/maven/org.apache.zipkin.proto3/zipkin-proto3/pom.properties > > > DISCLAIMER is missing, but I don't think CONTRIBUTING is in any way > required to be in the source dist.. a lot of it is irrelevant to a > standalone zip vs a repo root. Ex we don't contribute to a zip. > > Here's the disclaimer stuff > https://github.com/apache/incubator-zipkin-api/pull/65 > > I should have tipped off Jorge to do a REVIEW first to save some of > the recutting work by asking others to run the build like so and look > at the zip themselves GPG_TTY=$(tty) ./mvnw clean install > -Papache-release > > next time! > > -A > > On Fri, Apr 26, 2019 at 2:54 AM Zoltán Nagy <[email protected]> wrote: > > > > Sorry, I'm gonna be picky vote -1: the source archive (as well as the repo, > > though that's not as important) is missing the DISCLAIMER file. > > > > I also notice there are a bunch of files/folders that are in the git repo > > at commit 1e7797f72c5132d9ccc5612de949dc3f3fe312b6, but not in the source > > archive; can you confirm they're all intentional? (I'm not seeing an issue, > > just want to make sure): CONTRIBUTING.md, javadoc/, package.json, pom.xml, > > src/, validate.js > > > > The comment about artifact naming from zipkin-layout-factory applies here > > as well. > > > > Thank you for your work on setting up these releases! > > > > On Thu, Apr 25, 2019 at 4:02 PM Jorge Quilcate <[email protected]> > > wrote: > > > > > Hello Zipkin Community, > > > > > > This is a call for vote to release Apache Zipkin API (incubating) version > > > 0.2.1. > > > > > > The release candidates: > > > https://dist.apache.org/repos/dist/dev/incubator/zipkin/zipkin-api/0.2.1/ > > > > > > Git tag for the release: > > > https://github.com/apache/incubator-zipkin-api/tree/v0.2.1 > > > > > > Hash for the release tag: 1e7797f72c5132d9ccc5612de949dc3f3fe312b6 > > > > > > Release Notes: > > > https://github.com/apache/incubator-zipkin-api/releases/tag/v0.2.1 > > > > > > The artifacts have been signed with Key : 50D90C2C, which can be found in > > > the keys file: > > > https://dist.apache.org/repos/dist/dev/incubator/zipkin/KEYS > > > > > > Verification Hints: > > > For your convenience, the below includes detailed how-to on verifying a > > > source release. Please note that this document is a > > > work-in-progresshttps:// > > > cwiki.apache.org/confluence/display/ZIPKIN/Verifying+a+Source+Release > > > > > > The vote will be open for at least 72 hours or until necessary number of > > > votes are reached. > > > > > > Please vote accordingly: > > > > > > [ ] +1 approve > > > [ ] +0 no opinion > > > [ ] -1 disapprove with the reason > > > > > > Thanks, > > > The Apache Zipkin (Incubating) Team > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail:[email protected] > > > For additional commands, e-mail:[email protected] > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: [email protected] > > > For additional commands, e-mail: [email protected] > > > > > > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
