-
Notifications
You must be signed in to change notification settings - Fork 6
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
Funnannotate #426
Comments
Firs looking at DBD-mysql a depency and got stuck here. See WIP for the easyconfigs.
|
That looks like an incorrect check for the MySQL (MariaDB) version to me... It requires that the version starts with |
Maybe we should be using https://metacpan.org/dist/DBD-MariaDB instead? |
I just discovered that this exhists so did some double work https://github.com/easybuilders/easybuild-easyconfigs/blob/fdac40400e478c5bb4da1c7b6ef28376f8408f65/easybuild/easyconfigs/p/Perl-bundle-CPAN/Perl-bundle-CPAN-5.36.1-GCCcore-12.3.0.eb |
The documentation says that you have to install the actual MariaDB or MySQL as well |
with the previous version of DBD-mysql we also build with 10. But the check was not added in that version |
@laraPPr I guess we can patch the version check and see if that works? |
Yes I can try that |
I got that hook working but now it got stuck in the make step
|
Somethin else popped up with the dependency Trinity Most of its dependencies are build ontop of Java 11. Except for GATK which requires Java 17. Most of Trinity's dependencies I could build with Java 17 until I saw that it also uses R which is also build ontop of Java 11. I was than thinking maybe I should build on a newer toolchain but 2023b also uses Java11. |
Maybe the best option is just to build an older version of GATK that builds ontop of Java11 |
I tried building it with DBD::mariadb instead of DBD::mysql but that does not seem to work |
installing the new tbl2asn |
user is getting this error nextgenusfs/funannotate#1072. |
@boegel I think it is best to get easybuilders/easybuild-easyconfigs#21705 merged because it is only a single case that causes problems and there are now other open prs from @pavelToman that depend on it. |
easybuilders/easybuild-easyconfigs#21705 is merged, |
This can't be closed yet, more errors have been reported that we'll need to look into... |
foss/2023a
PythonBundle
funannotate check
to see missing dependenciesThe text was updated successfully, but these errors were encountered: