Skip to content

Conversation

@Kaweees
Copy link
Member

@Kaweees Kaweees commented Feb 12, 2026

This adds two missing library dependencies (gfortran.cc.lib and libidn2) to the flake configuration. These packages are needed to run on the Jetson platform. Related: #1229 and #980

@greptile-apps
Copy link

greptile-apps bot commented Feb 12, 2026

Greptile Overview

Greptile Summary

This PR adds two missing library dependencies to the Nix flake configuration:

  • gfortran.cc.lib: Required for scientific computing packages like scipy and numpy that need Fortran libraries for optimized linear algebra operations
  • libidn2: Required for internationalized domain name support in networking libraries

Both dependencies are correctly configured with ldLibraryGroup=true flags to ensure they're added to the LD_LIBRARY_PATH. The gfortran.cc.lib is placed alongside other compiler libraries in the "Core shell & utils" section, while libidn2 is placed in the "Open3D & build-time" section near other image/media libraries.

Confidence Score: 5/5

  • Safe to merge - adds only standard library dependencies without any code changes
  • The changes are minimal and low-risk, adding only two well-known library dependencies to the Nix flake. Both libraries are correctly configured with appropriate flags and placed in logical sections. No code logic is modified, only build configuration.
  • No files require special attention

Important Files Changed

Filename Overview
flake.nix Added gfortran.cc.lib and libidn2 library dependencies with correct flags

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@Kaweees Kaweees merged commit 45f38c8 into dev Feb 12, 2026
15 checks passed
@Kaweees Kaweees deleted the miguel/flake_update branch February 12, 2026 09:24
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.

2 participants