Its just a packing issue, we forgot in the package script to exclude those. Good find guys
Get Outlook for Android On Mon, Jun 24, 2019 at 7:00 PM +0100, "Heiser, Derek" <[email protected]> wrote: I would say that source directories shouldn't have /bin or /obj at all and can be added to .gitignore to prevent them from being committed. ~D -----Original Message----- From: Robbie Gemmell Sent: Monday, June 24, 2019 11:48 AM To: [email protected] Subject: Re: [VOTE] Release Apache NMS API 1.8.0 [External Email] ------------------------------------------------------------------------------ For the NOTICE, the date needs updating, the name should probably reference that its for the NMS API, the header is just junk and should be removed. The LICENCE file is seemingly full of a host of detail for Java-specific things that I doubt are shipped as part of the NMS API source release. It should reference only things relevent to the archive contents. I'm also seeing oddness around some dirs in the src archive (e.g. test/nms-api-test/bin/Debug and test/nms-api-test/obj/Debug) that upon extraction I dont have permission to. Viewing in the zip content listing suggests has further subdirs for different framework versions. Are these expected to be restricted like this, or present at all? (First time reviewing an NMS Robbie On Mon, 24 Jun 2019 at 17:29, wrote: > > So theyre the existing ones that were released in previous releases. > > > > > The only bit i just spotted is date needs updating. > > > > > Get Outlook for Android > > > > > > > > On Mon, Jun 24, 2019 at 5:17 PM +0100, "Robbie Gemmell" wrote: > > > > > > > > > > > The licence and notice files present in the src archive are in need of > updating (they look like they are from something else entirely, e.g > broker). > > On Mon, 24 Jun 2019 at 17:13, wrote: > > > > Thanks Tim for the feedback. Is that the only issue? Just to avoid too many > > respins. > > > > > > > > > > Get Outlook for Android > > > > > > > > > > > > > > > > On Mon, Jun 24, 2019 at 5:08 PM +0100, "Timothy Bish" wrote: > > > > > > > > > > > > > > > > > > > > > > -1 > > > > The docs and nuget package archives do not contain a proper license > > or notice files > > > > On 6/24/19 11:42 AM, Michael Pearce wrote: > > > Hi All, > > > > > > I have put together a spin for a Apache NMS API release, please > > > check it and vote accordingly. > > > > > > This release effectively updates the project to support .net standard 2.0. > > > > > > Also includes some modernisation of the project that was needed to > > > make the release, updating for latest visual studio, generating > > > new docs using docsfx, and lastly, creating a nuget package, that > > > once approved, we can publish to nuget. > > > > > > The files can be grabbed > > > from:https://dist.apache.org/repos/dist/dev/activemq/activemq-nms- > > > api/1.8.0-rc1/ > > > > > > The JIRAs assigned > > > are:https://issues.apache.org/jira/browse/AMQNET-588 > > > > > > https://issues.apache.org/jira/browse/AMQNET-585 > > > > > > > > > Regards, > > > Michael > > > > > > > -- > > Tim Bish > > > > > > > > > > > > > > > > > Follow Us: Facebook | Twitter | LinkedIn | YouTube
