Dependencies ( was: Re: [GUMP] Build Failure - jakarta-tomcat-5
When we add a new dependency to the code, we should: 1. Update gump 2. Make a small proposal and get a vote 3. Use conditionals if the dependency is not required and update download target Dependencies are very important and affect everyone. I don't remember when fileupload was added to HTMLManagerServlet - but it certainly doesn't belong to server/lib ( IMO it should go to WEB-INF/lib of the manager ). Costin [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/launcher/CatalinaLaunchFilter.java:69: > [package org.apache.commons.launcher does not exist javac] import > [org.apache.commons.launcher.LaunchCommand; > [javac]^ > [javac] [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/HTMLManagerServlet.java:86: > [package org.apache.commons.fileupload does not exist javac] import > [org.apache.commons.fileupload.FileItem; > [javac] ^ > [javac] > [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/HTMLManagerServlet.java:87: > [package org.apache.commons.fileupload does not exist javac] import > [org.apache.commons.fileupload.FileUpload; > [javac] ^ > [javac] > [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/HTMLManagerServlet.java:88: > [package org.apache.commons.fileupload does not exist javac] import > [org.apache.commons.fileupload.FileUploadException; > [javac] ^ > [javac] > [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/HTMLManagerServlet.java:198: > [cannot resolve symbol > [javac] symbol : class FileUpload > [javac] location: class > [org.apache.catalina.servlets.HTMLManagerServlet > [javac] FileUpload upload = new FileUpload(); > [javac] ^ > [javac] > [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/HTMLManagerServlet.java:198: > [cannot resolve symbol > [javac] symbol : class FileUpload > [javac] location: class > [org.apache.catalina.servlets.HTMLManagerServlet > [javac] FileUpload upload = new FileUpload(); > [javac] ^ > [javac] > [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/HTMLManagerServlet.java:209: > [cannot resolve symbol > [javac] symbol : class FileItem > [javac] location: class > [org.apache.catalina.servlets.HTMLManagerServlet > [javac] FileItem warUpload = null; > [javac] ^ > [javac] > [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/HTMLManagerServlet.java:216: > [cannot resolve symbol > [javac] symbol : class FileItem > [javac] location: class > [org.apache.catalina.servlets.HTMLManagerServlet > [javac] FileItem item = (FileItem) iter.next(); > [javac] ^ > [javac] > [/home/rubys/jakarta/jakarta-tomcat-catalina/catalina/src/share/org/apache/catalina/servlets/HTMLManagerServlet.java:216: > [cannot resolve symbol > [javac] symbol : class FileItem > [javac] location: class > [org.apache.catalina.servlets.HTMLManagerServlet > [javac] FileItem item = (FileItem) iter.next(); > [javac] ^ > [javac] Note: Some input files use or override a deprecated API. > [javac] Note: Recompile with -deprecation for details. > [javac] 12 errors > > BUILD FAILED > file:///home/rubys/jakarta/jakarta-tomcat-catalina/catalina/build.xml:652: > Compile failed; see the compiler error output for details. > > Total time: 35 seconds - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: Dependencies ( was: Re: [GUMP] Build Failure - jakarta-tomcat-5
Costin Manolache wrote: When we add a new dependency to the code, we should: 1. Update gump 2. Make a small proposal and get a vote 3. Use conditionals if the dependency is not required and update download target Dependencies are very important and affect everyone. I don't remember when fileupload was added to HTMLManagerServlet - but it certainly doesn't belong to server/lib ( IMO it should go to WEB-INF/lib of the manager ). The dependency was sneaked in by Glenn (and it is now predictably causing moral problems with 4.1.x; I will completely ignore these problems, and urge to proceed with the release if this is the only issue). I complained at the time, but the functionality appeared useful, so I decided not to veto his patch. Remy - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: Dependencies ( was: Re: [GUMP] Build Failure - jakarta-tomcat-5
Remy Maucherat wrote: > Costin Manolache wrote: >> When we add a new dependency to the code, we should: >> 1. Update gump >> 2. Make a small proposal and get a vote >> 3. Use conditionals if the dependency is not required and update download >> target >> >> Dependencies are very important and affect everyone. >> >> I don't remember when fileupload was added to HTMLManagerServlet - but it >> certainly doesn't belong to server/lib ( IMO it should go to WEB-INF/lib >> of the manager ). > > The dependency was sneaked in by Glenn (and it is now predictably > causing moral problems with 4.1.x; I will completely ignore these > problems, and urge to proceed with the release if this is the only issue). > I complained at the time, but the functionality appeared useful, so I > decided not to veto his patch. Well - I tought the rule is that we can't release with dependencies on unreleased or beta code. And HTMLManagerServlet was supposed to be a simple servlet - it now has dependencies on fileupload and that is forced on the whole thing. Glenn: is it ok if we comment this out for 4.1 ? Can you move the upload to a different servlet ? It doesn't have to be in HTMLManager, it can be in a regular standalone webapp. Costin - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: Dependencies ( was: Re: [GUMP] Build Failure - jakarta-tomcat-5
Costin Manolache wrote: Remy Maucherat wrote: Well - I tought the rule is that we can't release with dependencies on unreleased or beta code. And HTMLManagerServlet was supposed to be a simple servlet - it now has dependencies on fileupload and that is forced on the whole thing. Glenn: is it ok if we comment this out for 4.1 ? Can you move the upload to a different servlet ? It doesn't have to be in HTMLManager, it can be in a regular standalone webapp. Quite frankly, I feel lazy about the issue, and don't feel like redoing a full release because of that trivial issue. Remy - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
Re: Dependencies ( was: Re: [GUMP] Build Failure - jakarta-tomcat-5
Remy Maucherat wrote: > Costin Manolache wrote: >> Remy Maucherat wrote: >> >> Well - I tought the rule is that we can't release with dependencies on >> unreleased or beta code. >> >> And HTMLManagerServlet was supposed to be a simple servlet - it now has >> dependencies on fileupload and that is forced on the whole thing. >> >> Glenn: is it ok if we comment this out for 4.1 ? Can you move the upload >> to a different servlet ? It doesn't have to be in HTMLManager, it can be >> in a regular standalone webapp. > > Quite frankly, I feel lazy about the issue, and don't feel like redoing > a full release because of that trivial issue. ok. Costin - To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]