Appears there might be a few others places to update...
<snip>
[INFO]
------------------------------------------------------------------------
----
[INFO] Building Geronimo Maven2 Plugins :: Geronimo
[INFO] task-segment: [install]
[INFO]
------------------------------------------------------------------------
----
[INFO] [tools:require-java-version {execution: validate-java-version}]
[INFO] [plugin:descriptor]
[WARNING] Goal prefix is: geronimo; Maven currently expects it to be
geronimo
[INFO] Using 2 extractors.
[INFO] Applying extractor for language: java
[INFO] Extractor for language: java found 14 mojo descriptors.
[INFO] Applying extractor for language: bsh
[INFO] Extractor for language: bsh found 0 mojo descriptors.
[INFO] [tools:copy-legal-files {execution: install-legal-files}]
[INFO] Copying 2 files to /Users/jason/ws/geronimo/server/maven-
plugins/geronimo-maven-plugin/target/classes/META-INF
[INFO] [resources:resources]
[INFO] Using default encoding to copy filtered resources.
[INFO] [compiler:compile]
Compiling 26 source files to /Users/jason/ws/geronimo/server/maven-
plugins/geronimo-maven-plugin/target/classes
[INFO]
------------------------------------------------------------------------
[ERROR] BUILD FAILURE
[INFO]
------------------------------------------------------------------------
[INFO] Compilation failure
/Users/jason/ws/geronimo/server/maven-plugins/geronimo-maven-plugin/
src/main/java/org/apache/geronimo/mavenplugins/geronimo/server/
InstallerMojoSupport.java:[123,24] as of release 1.5, 'enum' is a
keyword, and may not be used as an identifier
(try -source 1.4 or lower to use 'enum' as an identifier)
Enumeration enum = zipFile.entries();
/Users/jason/ws/geronimo/server/maven-plugins/geronimo-maven-plugin/
src/main/java/org/apache/geronimo/mavenplugins/geronimo/server/
InstallerMojoSupport.java:[124,19] as of release 1.5, 'enum' is a
keyword, and may not be used as an identifier
(try -source 1.4 or lower to use 'enum' as an identifier)
while (enum.hasMoreElements()) {
/Users/jason/ws/geronimo/server/maven-plugins/geronimo-maven-plugin/
src/main/java/org/apache/geronimo/mavenplugins/geronimo/server/
InstallerMojoSupport.java:[125,43] as of release 1.5, 'enum' is a
keyword, and may not be used as an identifier
(try -source 1.4 or lower to use 'enum' as an identifier)
ZipEntry entry = (ZipEntry)enum.nextElement();
</snip>
--jason
On Nov 27, 2006, at 7:48 PM, anita kulshreshtha wrote:
Thanks Rick! Casting it to Class did the trick. The trunk (rev
479880) is now all 1.5!
Thanks
Anita
--- Rick McGuire <[EMAIL PROTECTED]> wrote:
anita kulshreshtha wrote:
I did not think that changing <source> from 1.4 to 1.5 will
cause
any problems. I tried it anyway.. The security module is giving
problem:
[INFO]
----------------------------------------------------------------------
--
[ERROR] BUILD FAILURE
[INFO]
----------------------------------------------------------------------
--
[INFO] Compilation failure
C:\anita\geronimo\g2.0\modules\geronimo-security\src\main\java\org
\apache\geronimo\securit
y\ContextManager.java:[256,32] cannot find symbol
symbol : method
getPrincipals
(java.lang.Class<org.apache.geronimo.security.realm.provider
s.GeronimoCallerPrincipal>)
location: class javax.security.auth.Subject
This compiles with source as 1.4. Any ideas?
Hmmm, for some reason, it's trying to use a generic there. Try
changing
the line to:
Set principals =
subject.getPrincipals((Class)GeronimoCallerPrincipal.class);
To explicitly get that to be a Class signature.
Rick
Thanks
Anita
--- Paul McMahan <[EMAIL PROTECTED]> wrote:
OK I misread your suggestion to say that the change should be made
"at
least on trunk". Now I realize you mean to change at least
<target>
if
not also <source> in the pom.xml. Why not change them both?
Paul
On 11/27/06, anita kulshreshtha <[EMAIL PROTECTED]> wrote:
IIUC, the branches/1.2 will always be 1.4 with an ability to
run
the
server in 1.5.
Thanks
Anita
--- Paul McMahan <[EMAIL PROTECTED]> wrote:
On 11/27/06, anita kulshreshtha <[EMAIL PROTECTED]> wrote:
I would like to change (at least) the target to 1.5 in the
trunk.
comments?
+1 on changing the target to 1.5 in trunk. I think the branches
should stay at 1.4 for now.
Best wishes,
Paul
______________________________________________________________________
______________
Do you Yahoo!?
Everyone is raving about the all-new Yahoo! Mail beta.
http://new.mail.yahoo.com
______________________________________________________________________
______________
Do you Yahoo!?
Everyone is raving about the all-new Yahoo! Mail beta.
http://new.mail.yahoo.com