[ 
https://issues.apache.org/jira/browse/DERBY-3986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12913049#action_12913049
 ] 

Knut Anders Hatlen commented on DERBY-3986:
-------------------------------------------

> So would the following make sense:
>
> i) Move cleangenerated logic out of clean into its own target
>
> ii) Tidy up the cleanparsers and cleanstate targets

Yes, makes sense. +1

> Of course, I could also eliminate the cleanparsers, cleantoursdb,
> cleancatalog, and cleanstate targets as you suggest. But these
> targets may still be useful. They may have been created to
> compensate for the spotty dependency tracking in our build
> scripts. What do you think?

I don't see much value in these targets myself, so unless someone
speaks up and says they actually use them regularly, I'd suggest that
we take this opportunity to trim down the size of the build
script. I'd rather add it back later if it turns out to be needed,
instead of keeping all the old cruft in case some of it turns out to
be useful.

> Stop dropping build artifacts in the subversion-controlled source tree
> ----------------------------------------------------------------------
>
>                 Key: DERBY-3986
>                 URL: https://issues.apache.org/jira/browse/DERBY-3986
>             Project: Derby
>          Issue Type: Improvement
>          Components: Build tools
>    Affects Versions: 10.5.1.1
>            Reporter: Rick Hillegas
>         Attachments: derby-3986-01-aa-checkCompilerLevel.diff, 
> derby-3986-02-af-sanity-bin-toursdb-storeless-release.diff, 
> derby-3986-02-ag-sanity-bin-toursdb-storeless-release.diff
>
>
> The Derby build drops various artifacts in the source tree. Build artifacts 
> should not muddy subversion controlled directories but should, instead, go 
> into temporary directories created by the the build. 
> The following is a (perhaps partial) list of artifacts currently dropped into 
> subversion-controlled space:
> Sanity directives
> Generated grammars
> The English error messages
> Data type class sizes
> Toursdb
> Temporary class for verifying compiler level

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to