Buildr
  1. Buildr
  2. BUILDR-333

buildr does not repackage when working with eclipse that compiles to target/classes

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: Wish List
    • Component/s: Compilers
    • Labels:
      None

      Description

      Many java developers set the project's output in Eclipse to target/classes to save compilation time of BuildR.

      Say I change a file:

      • jar task depends on compile.target
      • compile.target depends on compile task
      • compile task's needed? method checks Base#needed?
      • Base#needed? compares sources, classes and dependencies timestamps
        ==> since eclipse compiled the source, Base#needed? returns fals
        ==> nothing is done (jar is not repackaged)

      Suggestion: Jar task should compare jar and classes timestamp. if there are newer classes, it is needed to be executed

        Activity

        Hide
        Antoine Toulme added a comment -

        Targeting wishlist.

        Note Eclipse and Buildr compile and package in different ways: Eclipse will put resources next to .class files, while buildr puts resources in target/resources, that makes me think this is mostly either an invalid bug or a big problem.

        Show
        Antoine Toulme added a comment - Targeting wishlist. Note Eclipse and Buildr compile and package in different ways: Eclipse will put resources next to .class files, while buildr puts resources in target/resources, that makes me think this is mostly either an invalid bug or a big problem.
        Hide
        Ittay Dror added a comment -

        note that it looks like ArchiveTask#needed? is wrong anyway, since it will not recreate the archive if a file inside an included directory changes (since only the directory's timestamp will be checked)

        Show
        Ittay Dror added a comment - note that it looks like ArchiveTask#needed? is wrong anyway, since it will not recreate the archive if a file inside an included directory changes (since only the directory's timestamp will be checked)
        Hide
        Ittay Dror added a comment -

        Another suggestion is for ArchiveTask#needed? to be enhanced so it goes into folders to check if packaging is required

        Show
        Ittay Dror added a comment - Another suggestion is for ArchiveTask#needed? to be enhanced so it goes into folders to check if packaging is required

          People

          • Assignee:
            Unassigned
            Reporter:
            Ittay Dror
          • Votes:
            2 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development