On Sat, 28 Sep 2019 at 13:39, Gary Gregory <garydgreg...@gmail.com> wrote:
>
> It seems we provide too much in the src zip file like exe and obj files.
> Also old pdb files because I get:
>
> .\..\..\src\cmdline.c: fatal error C1051: program database file,
> 'C:\temp\rc\commons-daemon-1.2.2-src\src\native\windows\apps\prunsrv\WINXP_X86_EXE_RELEASE\prunsrv-src.pdb',
> has an obsolete format, delete it and recompile
> NMAKE : fatal error U1077: '"C:\Program Files (x86)\Microsoft Visual
> Studio\2019\Community\VC\Tools\MSVC\14.21.27702\bin\HostX86\x86\cl.EXE"' :
> return code '0x2'
>
> Since we do not provide class files and jars in the src zip, why should we
> provide pdb, objs and exes in these all caps folders?
>
> It seems we should NOT deliver the following folders in the src zip which
> now contain objs and exes:
>
> src\native\windows\apps\prunmgr\WINXP_X86_GUI_RELEASE
> src\native\windows\apps\prunsrv\WINXP_X86_EXE_RELEASE
> src\native\windows\apps\prunsrv\WINXP_X64_EXE_RELEASE
>
> WDYT?

Looks like the build created the files in the wrong directories?

The src zip also contains:
src/native/unix/configure
AIUI this is a generated file; I would expect it to be in the binary
artifact, if anywhere

Also there are some Git files missing from the src zip.
Not sure if that is intentional?

CONTRIBUTING.md
HOWTO-RELEASE.txt
README.md


> Gary
>
>
> On Wed, Sep 25, 2019 at 10:57 AM Mark Thomas <ma...@apache.org> wrote:
>
> > A further regression has been identified in the 1.2.0/1.2.1 releases so
> > I'd like to get 1.2.2 released to address it. So, time for another
> > release vote.
> >
> > Notable changes since 1.2.1 include:
> > - Correct a regression (DAEMON-408) that caused Windows services to
> >   crash on start-up when the universal C runtime was not available
> >
> > The full set of changes is in the changelog
> >
> > 1.2.2 RC1 can be obtained from (r36024)
> > https://dist.apache.org/repos/dist/dev/commons/daemon/
> >
> > The git tag is:
> > Tag: COMMONS_DAEMON_1_2_2_RC1
> > URL:
> >
> > https://gitbox.apache.org/repos/asf?p=commons-daemon.git;a=commit;h=cb6754d51a008418ed93e29cde141f046766fee3
> > Hash:  cb6754d51a008418ed93e29cde141f046766fee3
> >
> > The Maven Staging repo is:
> > https://repository.apache.org/content/repositories/orgapachecommons-1471/
> >
> > The Windows binaries have been signed by the DigiCert (formerly
> > Symantec) code signing service.
> >
> > Files signed with my preferred key:
> > http://people.apache.org/~markt/
> > KEYS file is standard Apache Commons keys file:
> > http://www.apache.org/dist/commons/KEYS
> >
> >
> > [ ] Approved - go ahead and release Commons Daemon 1.2.2 RC1 as 1.2.2
> > [ ] Broken   - do not release because...
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> > For additional commands, e-mail: dev-h...@commons.apache.org
> >
> >

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to