Skip to content

appflowy: 0.10.6 -> 0.11.4#497646

Open
DaVinci42 wants to merge 1 commit intoNixOS:masterfrom
DaVinci42:appflowy
Open

appflowy: 0.10.6 -> 0.11.4#497646
DaVinci42 wants to merge 1 commit intoNixOS:masterfrom
DaVinci42:appflowy

Conversation

@DaVinci42
Copy link
Contributor

@DaVinci42 DaVinci42 commented Mar 7, 2026

Diff: AppFlowy-IO/AppFlowy@0.10.6...0.11.3

Changelog: https://github.com/AppFlowy-IO/appflowy/releases/tag/0.11.3

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

@nixpkgs-ci nixpkgs-ci bot requested a review from DarkOnion0 March 7, 2026 17:32
@nixpkgs-ci nixpkgs-ci bot added 8.has: package (update) This PR updates a package to a newer version 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux. labels Mar 7, 2026
@DaVinci42
Copy link
Contributor Author

DaVinci42 commented Mar 7, 2026

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 497646
Commit: 79a1bd809a9be2de52b589c1af7676a779a29f7e (subsequent changes)
Merge: dcdd5b75c57b1ed374983f5fd2afedf0215aaed4

Logs: https://github.com/DaVinci42/nixpkgs-review-gha/actions/runs/22803824772

x86_64-linux

❌ 1 package failed to build:
  • appflowy

aarch64-darwin (sandbox = relaxed)

✅ 1 package built:
  • appflowy

@DaVinci42 DaVinci42 force-pushed the appflowy branch 5 times, most recently from 35ce73e to 8945ca6 Compare March 7, 2026 18:30
@DaVinci42
Copy link
Contributor Author

DaVinci42 commented Mar 7, 2026

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 497646
Commit: 8945ca62f02ae344410f951cefe423d5d4b519c3 (subsequent changes)
Merge: ed2e7ca7684f581ce2aa4ca71575eececb6fbc5c

Logs: https://github.com/DaVinci42/nixpkgs-review-gha/actions/runs/22804803942

x86_64-linux

✅ 1 package built:
  • appflowy

aarch64-darwin (sandbox = relaxed)

✅ 1 package built:
  • appflowy

DarkOnion0
DarkOnion0 reviewed Mar 8, 2026
View reviewed changes
Copy link
Contributor

@DarkOnion0 DarkOnion0 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

👍 Thanks for that PR (and sorry for not being super active updating appflowy). The app runs and builds nearly flawlessly.

The thing is that as for the previous releases, the audio embed is still broken, and I have mixed feelings about landing an update that doesn't fully work (even though I acknowledge it's been a while since appflowy has been updated in nixpkgs) 🫤. Moreover, the code on their repo has not been updated since roughly the end of Q4 2025, so I'm not sure if we can still maintain the current licence in the meta section of the package 🤷...

Lastly it would be better to just import libgbm instead of mesa. It is true that mesa contains libgbm, but it's a bit overkill.

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 497646
Commit: 8945ca62f02ae344410f951cefe423d5d4b519c3

x86_64-linux

✅ 2 packages built:
  • appflowy
  • nixpkgs-manual

@DaVinci42 DaVinci42 changed the title appflowy: 0.10.6 -> 0.11.3 appflowy: 0.10.6 -> 0.11.4 Mar 10, 2026
@DaVinci42
Copy link
Contributor Author

DaVinci42 commented Mar 10, 2026

👍 Thanks for that PR (and sorry for not being super active updating appflowy). The app runs and builds nearly flawlessly.

The thing is that as for the previous releases, the audio embed is still broken, and I have mixed feelings about landing an update that doesn't fully work (even though I acknowledge it's been a while since appflowy has been updated in nixpkgs) 🫤. Moreover, the code on their repo has not been updated since roughly the end of Q4 2025, so I'm not sure if we can still maintain the current licence in the meta section of the package 🤷...

Lastly it would be better to just import libgbm instead of mesa. It is true that mesa contains libgbm, but it's a bit overkill.

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 497646 Commit: 8945ca62f02ae344410f951cefe423d5d4b519c3

x86_64-linux

✅ 2 packages built:

thanks, updated

@DaVinci42
Copy link
Contributor Author

DaVinci42 commented Mar 10, 2026

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 497646
Commit: 1f318f843fa00556fdd05e2b8f0cfb2c676e7e37 (subsequent changes)
Merge: 6c76f191371dd0803c7cda28b5ca0f94fdfb168f

Logs: https://github.com/DaVinci42/nixpkgs-review-gha/actions/runs/22889692484

x86_64-linux

✅ 1 package built:
  • appflowy

x86_64-darwin (sandbox = relaxed)

✅ 1 package built:
  • appflowy

aarch64-darwin (sandbox = relaxed)

✅ 1 package built:
  • appflowy

DarkOnion0
DarkOnion0 reviewed Mar 12, 2026
View reviewed changes
Copy link
Contributor

@DarkOnion0 DarkOnion0 left a comment

Choose a reason for hiding this comment

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

Builds and runs fine, LGTM 👍 ! However, I will try to find information on what to do about the issues I mentioned earlier before going any further...

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 497646
Commit: 1f318f843fa00556fdd05e2b8f0cfb2c676e7e37

x86_64-linux

✅ 2 packages built:
  • appflowy
  • nixpkgs-manual

@DarkOnion0 DarkOnion0 mentioned this pull request Mar 12, 2026
Closed
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@DarkOnion0 DarkOnion0 DarkOnion0 left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

8.has: package (update) This PR updates a package to a newer version 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-darwin: 1 This PR causes 1 package to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-linux: 1 This PR causes 1 package to rebuild on Linux.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@DaVinci42 @DarkOnion0