cometd demo in Jetty 6.1.15 has hardcoded reference to ./target/cometd-demo-6.1-SNAPSHOT

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 6.1.16
  • Fix Version/s: 6.1.17
  • Component/s: Bayeux
  • Labels:
    None
  • Environment:
    Eclipse Ganymede, Sun Java 1.5, Fedora 10

Description

If I run the chat demo through maven, mvn jetty:run it works fine

If I try to import it into Eclipse and debug there, I run into
Exception in thread "main" java.lang.IllegalArgumentException: file:/home/ak/jetty-6.1.15/contrib/cometd/demo/target/cometd-demo-6.1-SNAPSHOT is not an existing directory.

This makes sense, at line 99 in CometdDemo.java there is indeed a hardcoded reference to -SNAPSHOT , replacing it with 6.1.15 solves the problem.

So if this is indeed an issue (not sure why mvn jetty:run works) , then it's probably because the script that replaces SNAPSHOT before a release does not get run on svn externals?

Thank you for great software

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Greg Wilkins added a comment -

yes indeed.

This is a problem.... still trying to think of a way to get around this hack for the test harness.
I think I need some more mavin magic.

Show
Greg Wilkins added a comment - yes indeed. This is a problem.... still trying to think of a way to get around this hack for the test harness. I think I need some more mavin magic.
Hide
Permalink
Greg Wilkins added a comment -

partial half hearted not so great fix applied. will see how it goes

Show
Greg Wilkins added a comment - partial half hearted not so great fix applied. will see how it goes

People

  • Assignee:
    Greg Wilkins
    Reporter:
    Arne Kepp
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: