Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.3.0
    • Component/s: contrib-bookkeeper
    • Labels:
      None

      Description

      The patch of ZOOKEEPER-507 requires netty and to compile we now have to manually add the netty jar to

      {project root}

      /src/java/lib. GIven that we bookkeeper already fetches junit and log4j using ivy, we could also do it for netty. However, we need to configure it to fetch from a different repository.

      1. zookeeper-633.patch
        4 kB
        Giridharan Kesavan
      2. zookeeper-633.patch
        5 kB
        Benjamin Reed
      3. zookeeper-633.patch
        6 kB
        Giridharan Kesavan
      4. ZOOKEEPER-633.patch
        6 kB
        Flavio Junqueira

        Activity

        Hide
        Giridharan Kesavan added a comment -

        This patch adds a new ivyconfiguration file called ivysettings.xml which has the details of the resolver.
        This new resolver would resolve dependencies from the jboss maven repo when the central maven repo
        doesn't have the required artifact.

        Show
        Giridharan Kesavan added a comment - This patch adds a new ivyconfiguration file called ivysettings.xml which has the details of the resolver. This new resolver would resolve dependencies from the jboss maven repo when the central maven repo doesn't have the required artifact.
        Hide
        Benjamin Reed added a comment -

        thanx giri! one small correction. netty has a type of "bundle" rather than "jar". (i'm not sure of the distinction, but i had to add a line to the ivy-retrieve to handle it.

        Show
        Benjamin Reed added a comment - thanx giri! one small correction. netty has a type of "bundle" rather than "jar". (i'm not sure of the distinction, but i had to add a line to the ivy-retrieve to handle it.
        Hide
        Flavio Junqueira added a comment -

        +1, It works great. The only small comment I have is that I'm getting the following message:

        [ivy:retrieve] DEPRECATED: 'ivy.conf.file' is deprecated, use 'ivy.settings.file' instead
        

        Does any of you know why I'm getting this message?

        Show
        Flavio Junqueira added a comment - +1, It works great. The only small comment I have is that I'm getting the following message: [ivy:retrieve] DEPRECATED: 'ivy.conf.file' is deprecated, use 'ivy.settings.file' instead Does any of you know why I'm getting this message?
        Hide
        Giridharan Kesavan added a comment -

        this patch fixes the DEPRECATED warning message.

        tnx

        Show
        Giridharan Kesavan added a comment - this patch fixes the DEPRECATED warning message. tnx
        Hide
        Flavio Junqueira added a comment -

        Although the latest patch gets rid of the deprecated message, it does not include the bundle ivy:retrieve option, so I'm uploading a new patch that takes care of it.

        Show
        Flavio Junqueira added a comment - Although the latest patch gets rid of the deprecated message, it does not include the bundle ivy:retrieve option, so I'm uploading a new patch that takes care of it.
        Hide
        Flavio Junqueira added a comment -

        Thanks, Giri and Ben, I have just committed this one (Committed revision 896044).

        Show
        Flavio Junqueira added a comment - Thanks, Giri and Ben, I have just committed this one (Committed revision 896044).
        Hide
        Hudson added a comment -

        Integrated in ZooKeeper-trunk #655 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/655/)

        Show
        Hudson added a comment - Integrated in ZooKeeper-trunk #655 (See http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/655/ )

          People

          • Assignee:
            Giridharan Kesavan
            Reporter:
            Flavio Junqueira
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development