IvyDE
  1. IvyDE
  2. IVYDE-27

let IvyDE refresh workspace after a resolve

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.2.0.beta1
    • Component/s: classpath container
    • Labels:
      None
    • Environment:

      any

      Description

      Hi, it would be great if IvyDE could refresh the workspace after a resolve. For example, the web tools complain that files have changed if I don't refresh manually. (as a related matter, would it be possible to have IvyDE as a builder so it could be run for every build and didn't need to be started manually?) Cheers, Alex

      1. IvyResolveJobThread-patch.java
        1 kB
        Clint Burghduff
      2. IVYDE-27.zip
        14 kB
        Clint Burghduff

        Activity

        Hide
        Clint Burghduff added a comment -

        The attached file contains a patch for the IvyResolveJobThread.java. It fixes the issue with the project not getting refreshed after a retrieve. I also included an ivydetest-retrieve-after-resolve-refresh folder which shows the lib folder getting created after doing a resolve/retrieve.

        Show
        Clint Burghduff added a comment - The attached file contains a patch for the IvyResolveJobThread.java. It fixes the issue with the project not getting refreshed after a retrieve. I also included an ivydetest-retrieve-after-resolve-refresh folder which shows the lib folder getting created after doing a resolve/retrieve.
        Hide
        Nicolas Lalevée added a comment -

        Your patch seem ok.
        It would be not backward compatible in term of functionality though. People still wanting no refresh on resolve would be forced to. A new configuration option would be nice here.

        Show
        Nicolas Lalevée added a comment - Your patch seem ok. It would be not backward compatible in term of functionality though. People still wanting no refresh on resolve would be forced to. A new configuration option would be nice here.
        Hide
        Clint Burghduff added a comment -

        Nicolas,

        Thanks for taking a look at this. Below was my thinking when I made the change. If you still think there should be an option, I can look at what it would take to implement that.

        I was thinking that if they have the "Do retrieve after Resolve" setting selected, then it should do a refresh so that the IDE matches what is on the file system. I was looking at this more as a Bug, not an Improvement.

        Thanks
        Clint

        Show
        Clint Burghduff added a comment - Nicolas, Thanks for taking a look at this. Below was my thinking when I made the change. If you still think there should be an option, I can look at what it would take to implement that. I was thinking that if they have the "Do retrieve after Resolve" setting selected, then it should do a refresh so that the IDE matches what is on the file system. I was looking at this more as a Bug, not an Improvement. Thanks Clint
        Hide
        Nicolas Lalevée added a comment -

        I see your point Clint.

        I looked again to your patch, it seems refresh the entire project, right ? I think this can be an issue for project which have a huge amount of files (I do have one with more than 10000 files, .svn included).

        So I suggest to either add an option, or make IvyDE smarter to only refresh the files where retrieved into.

        Show
        Nicolas Lalevée added a comment - I see your point Clint. I looked again to your patch, it seems refresh the entire project, right ? I think this can be an issue for project which have a huge amount of files (I do have one with more than 10000 files, .svn included). So I suggest to either add an option, or make IvyDE smarter to only refresh the files where retrieved into.
        Hide
        Clint Burghduff added a comment -

        Updated patch file that will only refresh from the root of the ivy pattern specified.

        Show
        Clint Burghduff added a comment - Updated patch file that will only refresh from the root of the ivy pattern specified.
        Hide
        Nicolas Lalevée added a comment -

        Thanks for the patch Clint ! I didn't knew it was that easy to find the root of the pattern

        Show
        Nicolas Lalevée added a comment - Thanks for the patch Clint ! I didn't knew it was that easy to find the root of the pattern

          People

          • Assignee:
            Nicolas Lalevée
            Reporter:
            Alexander Voss
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development