[jira] Commented: (LUCENE-908) MANIFEST.MF cleanup (main jar and luci customizations)
[ https://issues.apache.org/jira/browse/LUCENE-908?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12502040 ] Michael Busch commented on LUCENE-908: -- * manifest file in any of gdata's jars/war (it doesn't use the contrib-build.xml either) * should luci's Class-Path refer to the full name of the lucene core jar? I would like to ask the contrib owners to take care of these issues. * spec version must match digit+{.digit+}* ... this is true for our official releases, but broken in our nightlies. I will leave this for now as this patch doesn't change the spec version. * need to svn remove the existing luci MANIFEST file * manifest file in demo war file Will take care... MANIFEST.MF cleanup (main jar and luci customizations) -- Key: LUCENE-908 URL: https://issues.apache.org/jira/browse/LUCENE-908 Project: Lucene - Java Issue Type: Bug Components: Build Reporter: Michael Busch Assignee: Michael Busch Priority: Trivial Fix For: 2.2 Attachments: LUCENE-908.patch there are several problems with the MANIFEST.MF file used in the core jar, and some inconsistencies in th luci jar: Lucli's build.xml has an own jar target and does not use the jar target from common-build.xml. The result is that the MANIFEST.MF file is not consistent and the META-INF dir does not contain LICENSE.TXT and NOTICE.TXT. Is there a reason why lucli behaves different in this regard? If not I think we should fix this. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[jira] Commented: (LUCENE-908) MANIFEST.MF cleanup (main jar and luci customizations)
[ https://issues.apache.org/jira/browse/LUCENE-908?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12502104 ] Hoss Man commented on LUCENE-908: - A couple of random thoughts... 1) macro's can take multiple optional named element tags to embed in their bodies ... so instead of declaring a refid for what to include in the metinf, callers of the macro could put metainf call directly in the call to jarify 2) one way to reduce some redundancy in the build files (between jar-core, jar-demo, and war-demo) might be to use the manifest task instead of the manifest sub element of the jar task ... there are a few subtle differences but the main key is that the manifest taks let's you build up a file which you can then refer to by name from the jar task ... we could have a single buildmanifest macro with all of the common attributes in it and then it could be called from the various jar/war targets just before building the actual jar using attributes and element tags to customize things that need to be different. ...neither of these are crucial, they're just things you might want to consider to keep the build files smaller (and arguably simpler) MANIFEST.MF cleanup (main jar and luci customizations) -- Key: LUCENE-908 URL: https://issues.apache.org/jira/browse/LUCENE-908 Project: Lucene - Java Issue Type: Bug Components: Build Reporter: Michael Busch Assignee: Michael Busch Priority: Trivial Fix For: 2.2 Attachments: lucene-908.patch, LUCENE-908.patch there are several problems with the MANIFEST.MF file used in the core jar, and some inconsistencies in th luci jar: Lucli's build.xml has an own jar target and does not use the jar target from common-build.xml. The result is that the MANIFEST.MF file is not consistent and the META-INF dir does not contain LICENSE.TXT and NOTICE.TXT. Is there a reason why lucli behaves different in this regard? If not I think we should fix this. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[jira] Commented: (LUCENE-908) MANIFEST.MF cleanup (main jar and luci customizations)
[ https://issues.apache.org/jira/browse/LUCENE-908?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12502156 ] Michael Busch commented on LUCENE-908: -- As always these are very good recommendations Hoss! I think I will commit my patch for 2.2, because it works fine. But I will leave this issue open (just clear the Fix version) to keep in mind that we want to make these improvements. MANIFEST.MF cleanup (main jar and luci customizations) -- Key: LUCENE-908 URL: https://issues.apache.org/jira/browse/LUCENE-908 Project: Lucene - Java Issue Type: Bug Components: Build Reporter: Michael Busch Assignee: Michael Busch Priority: Trivial Fix For: 2.2 Attachments: lucene-908.patch, LUCENE-908.patch there are several problems with the MANIFEST.MF file used in the core jar, and some inconsistencies in th luci jar: Lucli's build.xml has an own jar target and does not use the jar target from common-build.xml. The result is that the MANIFEST.MF file is not consistent and the META-INF dir does not contain LICENSE.TXT and NOTICE.TXT. Is there a reason why lucli behaves different in this regard? If not I think we should fix this. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[jira] Commented: (LUCENE-908) MANIFEST.MF cleanup (main jar and luci customizations)
[ https://issues.apache.org/jira/browse/LUCENE-908?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12501713 ] Michael Busch commented on LUCENE-908: -- Hi Hoss, I think this makes sense. It would be great if you could provide a patch here? MANIFEST.MF cleanup (main jar and luci customizations) -- Key: LUCENE-908 URL: https://issues.apache.org/jira/browse/LUCENE-908 Project: Lucene - Java Issue Type: Bug Components: Build Reporter: Michael Busch Assignee: Michael Busch Priority: Trivial Fix For: 2.2 there are several problems with the MANIFEST.MF file used in the core jar, and some inconsistencies in th luci jar: Lucli's build.xml has an own jar target and does not use the jar target from common-build.xml. The result is that the MANIFEST.MF file is not consistent and the META-INF dir does not contain LICENSE.TXT and NOTICE.TXT. Is there a reason why lucli behaves different in this regard? If not I think we should fix this. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
[jira] Commented: (LUCENE-908) MANIFEST.MF cleanup (main jar and luci customizations)
[ https://issues.apache.org/jira/browse/LUCENE-908?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12501798 ] Michael Busch commented on LUCENE-908: -- Michael i'm hoping you can take the ball and run with it, Thanks for the pass, Hoss, I'm already running... MANIFEST.MF cleanup (main jar and luci customizations) -- Key: LUCENE-908 URL: https://issues.apache.org/jira/browse/LUCENE-908 Project: Lucene - Java Issue Type: Bug Components: Build Reporter: Michael Busch Assignee: Michael Busch Priority: Trivial Fix For: 2.2 Attachments: LUCENE-908.patch there are several problems with the MANIFEST.MF file used in the core jar, and some inconsistencies in th luci jar: Lucli's build.xml has an own jar target and does not use the jar target from common-build.xml. The result is that the MANIFEST.MF file is not consistent and the META-INF dir does not contain LICENSE.TXT and NOTICE.TXT. Is there a reason why lucli behaves different in this regard? If not I think we should fix this. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]