Return-Path: X-Original-To: apmail-camel-dev-archive@www.apache.org Delivered-To: apmail-camel-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id D6EC8FA4B for ; Mon, 29 Apr 2013 06:06:36 +0000 (UTC) Received: (qmail 62384 invoked by uid 500); 29 Apr 2013 06:06:36 -0000 Delivered-To: apmail-camel-dev-archive@camel.apache.org Received: (qmail 62215 invoked by uid 500); 29 Apr 2013 06:06:36 -0000 Mailing-List: contact dev-help@camel.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@camel.apache.org Delivered-To: mailing list dev@camel.apache.org Received: (qmail 62203 invoked by uid 99); 29 Apr 2013 06:06:36 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 29 Apr 2013 06:06:36 +0000 X-ASF-Spam-Status: No, hits=0.6 required=5.0 tests=RCVD_IN_DNSWL_LOW,SPF_PASS,URI_HEX X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of james.strachan@gmail.com designates 209.85.220.177 as permitted sender) Received: from [209.85.220.177] (HELO mail-vc0-f177.google.com) (209.85.220.177) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 29 Apr 2013 06:06:32 +0000 Received: by mail-vc0-f177.google.com with SMTP id gf12so2961735vcb.22 for ; Sun, 28 Apr 2013 23:06:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:mime-version:in-reply-to:references:from:date:message-id :subject:to:content-type; bh=bnl/0O2tSHEoq2RiA6Vzo0aO2Y3fquAkG9hDS+EAudw=; b=qKcn594d6vsolgH7XKQDJZ+czKLrL3gcu+CqO72I2koQrli4XPRW0q2FfkgouBNfqW f6k06KnVuSb88wAzSDih2hYHaLVd22yBHkwHC3vk4SqHFwSVfka3Mbt2AYDg/60DAcYg 3upLNbfA9KWXDX+9fBZnPJM6A3OvKdmucUKhLJ6StOrwnLiDcmd0SArg4oSIk2XO/mik Cdbc7UPFghcoPDLaQbD1AolG4O4vtXCLTHkRBwSBoTnX0mHjIUt7smPtz+WdJBOIZ8ka jFM7X3AfVEckKso2W4O+8wCrj7TeL6hVuPxNnUPrcOQ8PoWu+AjwHAuuAoDXgzeqFH2/ OuvA== X-Received: by 10.58.90.66 with SMTP id bu2mr32667262veb.29.1367215571453; Sun, 28 Apr 2013 23:06:11 -0700 (PDT) MIME-Version: 1.0 Received: by 10.59.13.163 with HTTP; Sun, 28 Apr 2013 23:05:51 -0700 (PDT) In-Reply-To: References: <1367189188134-5731708.post@n5.nabble.com> From: James Strachan Date: Mon, 29 Apr 2013 07:05:51 +0100 Message-ID: Subject: Re: improving Endpoint documentation and tooling To: dev@camel.apache.org Content-Type: text/plain; charset=ISO-8859-1 X-Virus-Checked: Checked by ClamAV on apache.org On 29 April 2013 07:00, James Strachan wrote: > On 28 April 2013 23:46, Babak Vahdat wrote: >> Hi James, >> >> Another awesome stuff of yours, Thanks! >> >> Though I've got three questions: >> >> 1) Following the instructions of yours above I don't see any generated html >> docs @ >> >> camel-core/target/classes/org/apache/camel/component >> >> Is there any other steps I'm missing here? > > No - though I used JDK 1.7; which JDK & OS are you using? FWIW I just tried JDK 1.6 then I got no HTML generated. The build seems to work OK for me though. >> 2) My suspicion is that the recent failed compilation by our >> Camel.trunk.notest CI-Server profile is because of the new Maven module >> (org.apache.camel.apt) as apparently JDK 6 annotation processing has known >> bug(s). So my question is if there would be some easy way to disable >> annotation processing (e.g using a profile) so that we could experiment this >> on CI-Server to see if this would be really the root cause of the problem? > > Yeah, we could disable the apt dependency unless on JDK. Am on > vacation today; will take a stab tomorrow if noone beats me to it. The > apt is only for the doc generation; so its no biggie to only enable it > on a profile or jdk basis. > > > >> And if we could provide a proper fix of this. More details here: >> >> >> http://camel.465427.n5.nabble.com/Our-builds-looks-really-bad-td5731673.html >> >> 3) Would it make sense to restrict the EndpointAnnotationProcessor's >> @SupportedAnnotationTypes to ONLY: >> >> @SupportedAnnotationTypes({"org.apache.camel.spi.*"}) >> >> I think in this way we could skip the non relevant annotations we have like >> "org.apache.camel.language.NamespacePrefix" which is currently causing >> problem on CI-Server. > > Ah yeah, thats a good idea! Just committed; I also used Java 7 source > format for the annotation processor in case that helps. I just rolled that bit back; as the Java 7 source flag doesn't work on JDK 1.6 :) -- James ------- Red Hat Email: jstracha@redhat.com Web: http://fusesource.com Twitter: jstrachan, fusenews Blog: http://macstrac.blogspot.com/ Open Source Integration