Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for Amazon failing. #1599

Merged
merged 1 commit into from
Oct 11, 2024
Merged

Fix for Amazon failing. #1599

merged 1 commit into from
Oct 11, 2024

Conversation

lbergelson
Copy link
Contributor

@jrobinso Can you test this on your system before we merge? It seems to fix the problem for me but I want to be sure it's also fixing it for you.

* Fix the amazon failures due to LogFactory not found by adding a module dependency
* on apache logging.  This is necessary because commons-logging became modularized
* between 1.2.0 and 1.3.0 but amazon is expecting the non-modular 1.2.0 while
* 1.3.0 is being brought in due to changes in htsjdk.
* See https://logging.apache.org/blog/2023/12/02/apache-common-logging-1.3.0.html for
* more info.
* Fix for #1598
@jrobinso jrobinso merged commit c0e7f09 into main Oct 11, 2024
2 checks passed
@jrobinso jrobinso deleted the lb_fix_amazon branch October 11, 2024 15:31
@brainstorm
Copy link
Contributor

@jrobinso Please can you expedite this fix and cut a new IGV release ASAP? Our team is currently experiencing some issues that seem to point to this.

/cc @ohofmann @victorskl

@jrobinso
Copy link
Contributor

We're targetting a release for the end of the month. In the meantime I suggest you use 2.18.1. Alternatively you could use the "snapshot" build, in fact it would be useful to know if that build resolves your issues. If it doesn't then cutting a new release will not either. You can install it from https://igv.org/doc/desktop/#DownloadSnapshot/

@brainstorm
Copy link
Contributor

Thanks Jim, we had to ship a fix at our data portal frontend (outside of IGV) as a workaround so we're good for now, we'll potentially revert that change and test out the new release or test with our dev environment.

@brainstorm
Copy link
Contributor

brainstorm commented Nov 27, 2024

Jim, tested on our dev environment and indeed the snapshot build 2024-11-22 seems to fix the issue.

I'll circulate the snapshot release now to our curators for now, but I'd prefer to have a point release for easier tracking of changes and compliance.

@jrobinso
Copy link
Contributor

jrobinso commented Nov 27, 2024 via email

@brainstorm
Copy link
Contributor

Thanks Jim, very much appreciated!!!

@jrobinso
Copy link
Contributor

We're having trouble with Mac notarization, that won't get resolved before the holidays. Links below for Windows and the generic command line build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants