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

Reply via email to