Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Gabriel Selzer
    @gselzer
    @ctrueden I think as far as scijava packages rules go, it does probably make sense to move them into scijava-maven-plugin
    7 replies
    Unless you see someone wanting to use the rules without the rest of the stuff in that repo
    2 replies
    I don't know enough about that repo to make that call\
    Gabriel Selzer
    @gselzer
    @ctrueden my scijava-maven-plugin build is failing locally
    Curtis Rueden
    @ctrueden
    Failing build? Or failing tests?
    Gabriel Selzer
    @gselzer
    Failing integration test
    [INFO] [INFO] Compiling 1 source file to /mnt/c/Code/scijava/scijava-maven-plugin/target/it/populate-app/target/classes
    [INFO] [INFO] -------------------------------------------------------------
    [INFO] [ERROR] COMPILATION ERROR :
    [INFO] [INFO] -------------------------------------------------------------
    [INFO] [ERROR] Source option 5 is no longer supported. Use 6 or later.
    [INFO] [ERROR] Target option 1.5 is no longer supported. Use 1.6 or later.
    [INFO] [INFO] 2 errors
    [INFO] [INFO] -------------------------------------------------------------
    [INFO] [INFO] ------------------------------------------------------------------------
    [INFO] [INFO] BUILD FAILURE
    [INFO] [INFO] ------------------------------------------------------------------------
    [INFO] [INFO] Total time:  1.568 s
    [INFO] [INFO] Finished at: 2021-11-16T12:17:37-06:00
    [INFO] [INFO] ------------------------------------------------------------------------
    [INFO] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.1:compile (default-compile) on project Example_PlugIn: Compilation failure: Compilation failure:
    [INFO] [ERROR] Source option 5 is no longer supported. Use 6 or later.
    [INFO] [ERROR] Target option 1.5 is no longer supported. Use 1.6 or later.
    [INFO] [ERROR] -> [Help 1]
    Curtis Rueden
    @ctrueden
    Blech.
    You are building with a too-new version of Java. Use Java 8.
    Gabriel Selzer
    @gselzer
    Hmm, Java 8 supports Java 5 source/target?
    Curtis Rueden
    @ctrueden
    • 8: 1.2
    • 11: 6
    • 17: 7
    (Version: minimum source/target version)
    Gabriel Selzer
    @gselzer
    Aha
    Should we make an issue to update?
    Or maybe SciJava Maven Plugin2 can enter the incubator :sweat_smile:
    Curtis Rueden
    @ctrueden
    I don't understand....
    Gabriel Selzer
    @gselzer
    Do we not want to migrate all SciJava code to Java 11?
    Curtis Rueden
    @ctrueden
    Nope!
    Gabriel Selzer
    @gselzer
    What is your reasoning?
    Curtis Rueden
    @ctrueden
    scijava-maven-plugin is part of the current generation of libraries we support, and is built on java 8.
    It is weird that you get that error about 1.5.
    I don't see anywhere in the scijava-maven-plugin code where source or target of 1.5 is being passed!
    Gabriel Selzer
    @gselzer

    It is weird that you get that error about 1.5.

    Yeah, I couldn't find that either

    Curtis Rueden
    @ctrueden
    Ah, it's probably just because some IT doesn't extend pom-scijava.
    Gabriel Selzer
    @gselzer
    Building with Java 8 fixed it though
    Curtis Rueden
    @ctrueden
    The mvn default is 1.5.
    :+1:
    Gabriel Selzer
    @gselzer

    Ah, it's probably just because some IT doesn't extend pom-scijava.

    It's this fella

    Curtis Rueden
    @ctrueden
    Right, because it doesn't extend pom-scijava.
    Gabriel Selzer
    @gselzer
    I'll fix it and file a PR
    Curtis Rueden
    @ctrueden
    I don't think it needs fixing.
    Just use Java 8 for now.
    I don't feel strongly... we could have all the ITs extend pom-scijava... but I'm not sure!
    Gabriel Selzer
    @gselzer
    I don't feel strongly... we could have all the ITs extend pom-scijava... but I'm not sure!
    It seems strange that only one IT doesn't extend it
    Oh wait, many don't extend it
    I'm surprised only one failed...
    Jan Eglinger
    @imagejan
    @ctrueden @hinerm is there a chance we can get scijava/scijava-common#400 and scijava/scijava-ui-swing#51 merged and released soon? They’re relatively small changes, but I just stumbled again upon the non-working Enum script parameters (scijava/scijava-common#397) and those PRs should fix it in my view.
    9 replies
    Jan Eglinger
    @imagejan
    maven.scijava.org seems to be down again?
    and fine again.
    2 replies
    John Bogovic
    @bogovicj
    maven.scijava.org down again?
    Jan Eglinger
    @imagejan
    hm, yes, it seems so. @ctrueden?
    Curtis Rueden
    @ctrueden
    Yeah, I'm on it, but instead of papering over the problem this time, I'm digging a bit to better understand what's happening.
    FWIW, I also have the upgraded drives in hand, and plan to do that upgrade this coming weekend.
    John Bogovic
    @bogovicj
    :+1: thanks @ctrueden !
    Curtis Rueden
    @ctrueden
    Nexus is restarting now. Sorry for taking so long to do it.
    John Bogovic
    @bogovicj
    no prob, it's back for me!
    Jean-Yves Tinevez
    @tinevez
    There was a bug yesterday during the build and the latest TrackMate 7.5.0 did not make it there. Any idea at how can I fix this?
    19 replies
    Ulrik Günther
    @skalarproduktraum
    hey everyone! quick question on commands: I'd like to have a dropdown of the open images in fiji, how do i declare this?
    (also -- somewhat belated -- a very happy new year to all of you!)