Skip to content

Conversation

@eunos-1128
Copy link
Collaborator

This pull request makes several improvements to the Flatpak packaging and build process for the project, focusing on updating dependency handling, patch management, and installation steps. It also updates documentation to clarify Flatpak installation commands. The changes improve maintainability and ensure that the build uses the correct versions and locations for dependencies and patches.

Flatpak build and dependency management:

  • Changed the clipper4coot module name to clipper in io.github.pemsley.coot.yaml to better reflect the dependency's actual name.
  • Updated patch file paths for dependencies to reside in the patches/ directory, improving organization and clarity. [1] [2] [3]
  • Updated the Coot source reference to use the Release-1.1.19 tag instead of a specific commit, ensuring builds use a stable, versioned release.

Build process and installation improvements:

  • Removed an unnecessary mkdir -p command in the post-install step, as the subsequent cp -r command creates the directory if needed.
  • Added a patch (patches/Makefile.am.patch) to include the monomers directory in the build, ensuring all necessary resources are packaged.

Documentation updates:

  • Updated Flatpak installation instructions in README.md to include the --user flag, clarifying that the remote is added for the current user only. [1] [2]

@eunos-1128 eunos-1128 self-assigned this Oct 9, 2025
@flathubbot
Copy link

🚧 Test build enqueued.

@flathubbot
Copy link

🚧 Started test build.

@flathubbot
Copy link

🚧 Test build enqueued.

@flathubbot
Copy link

Test build was cancelled.

@flathubbot
Copy link

🚧 Started test build.

@flathubbot
Copy link

Test build failed.

@flathubbot
Copy link

Test build succeeded. To test this build, install it from the testing repository:

flatpak install --user https://dl.flathub.org/build-repo/217665/io.github.pemsley.coot.flatpakref

@eunos-1128 eunos-1128 merged commit c0883f6 into flathub:alpha Oct 9, 2025
1 check passed
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