ant-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 49473] New: [PATCH] missing extension-point behaviour
Date Sat, 19 Jun 2010 23:08:24 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=49473

           Summary: [PATCH] missing extension-point behaviour
           Product: Ant
           Version: 1.8.1
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: enhancement
          Priority: P2
         Component: Core
        AssignedTo: notifications@ant.apache.org
        ReportedBy: danny@codeaholics.org


Created an attachment (id=25621)
 --> (https://issues.apache.org/bugzilla/attachment.cgi?id=25621)
Patch for requested enhancement

Hi guys,

I'm loving the new 1.8 extension point functionality.

I'm working on a project which has multiple deployable components ("services")
and has separate build and deploy scripts. I'd like to use extension points to
be able to write a single "descriptor" for any given service in order to
centralise the definitions of services which are currently sprinkled liberally
throughout the build and deploy scripts.

The problem I have is that some of the targets in the descriptor are extensions
of extension points in the build script (and are only used during build) and
some are extensions of extension points in the deploy script (and are only used
during deployment). Of course, this doesn't work because ProjectHelper2 tries
to verify the existence of an extension point during project file parsing and
which extension points are present is a function of whether the importing
script is the build script or the deploy script.

I've made some changes to add an "extension-point-missing" attribute which
takes values:
- "error" - the current behaviour (and default if the attribute is missing)
- "warn" - log a warning and ignore (see next)
- "ignore" - don't stop or warn, and don't add the extension target as a
dependency of the (non-existent) extension point

I've attached a patch for consideration for Ant 1.8.2

Thanks

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Mime
View raw message