Skip to content

Conversation

@APCCV
Copy link
Contributor

@APCCV APCCV commented Nov 7, 2024

Maintainer: Antonio Pastor / @APCCV
Compile tested: ipq806x SDK (23.05.3, snapshot)
Run tested: ipq806x - C2600: start server, connect from MacOS, read/write files to home share, create/update TimeMachine backups. File sharing from MacOS 9.2.

Description: Backport of Netatalk to 23.05. Cherry-picked commits:

  • 645b04c: update to 3.2.0 restore from abandoned packages
  • be5ca7d: update to 3.2.5
  • d0abe3d: update to 4.0.4 using meson build
  • 049a055: Makefile cleanup for 4.0.4
  • 09a2427: AppleTalk feature on 4.0.4
  • 07efd50: glib2 dependency fix for 4.0.4
  • c487146: update to 4.0.7

AppleTalk feature tested fully working under 23.05 (provided a separate PR for kernel module is merged too).

@APCCV
Copy link
Contributor Author

APCCV commented Nov 7, 2024

Hmmm.... @Neustradamus we might be too late for this (is 23.05 is still being built?)
Hopefully the failed checks are just due to buildbot upgrades and this can still be backported. This would provide netatalk on stable build.

@APCCV APCCV changed the title netatalk: Update to 4.0.4 (backport) [23.05] netatalk: Update to 4.0.4 (backport) Nov 7, 2024
@APCCV APCCV marked this pull request as draft November 7, 2024 15:33
@APCCV APCCV marked this pull request as ready for review November 7, 2024 15:34
@Neustradamus
Copy link

@APCCV: 23.05.x is always good, not EOL yet.

@APCCV
Copy link
Contributor Author

APCCV commented Nov 14, 2024

Took out last two commits enabling AppleTalk as more work is required to get it fully working. The previous 4 commits are solid and provide netatalk support for 23.05.

I get that servers are failing CI checks because they are being upgraded... any chance we can merge so this package becomes available in 23.05 while upgrades are in progress? These are 4 cherry-picked commits that build fine in 24.10 and snapshot.

@APCCV
Copy link
Contributor Author

APCCV commented Nov 23, 2024

There we go. All green.
It's not easy being green.

@Neustradamus
Copy link

@APCCV: Good job!
All the users community will be happy!

@APCCV
Copy link
Contributor Author

APCCV commented Nov 23, 2024

@Neustradamus Ha. Nobody but you and I have seen this PR yet. Don't get too excited.

@mhei mhei added the OpenWrt 23.05 (end of support) Issue/PR on branch 23.05 label Nov 30, 2024
@APCCV
Copy link
Contributor Author

APCCV commented Dec 6, 2024

@ OpenWrt maintainers: please merge with 23.05. AppleTalk functionality has issues with DSA images but works beautifully with 23.05. It might be the case that 23.05 is the last release that supports AppleTalk. It would be really awesome if we can get AppleTalk in the last service release of 23.05.

Note that applies only to AppleTalk feature. AFPD (Apple File Sharing through IP) works great in 23.05, 24.10 & snapshot.

@APCCV
Copy link
Contributor Author

APCCV commented Dec 6, 2024

@BKPepe - apologies for the ping... but I would hate to miss the opportunity to get out a fully working AppleTalk daemon on an OpenWrt release. Full disclosure - for that to happen I will also need to PR a change to openwrt (not packages) so kmod-appletalk is also included in 23.05. I will submit that PR once this one is merged.
Even if that second PR for backport is not accepted there is still value with a working AFP daemon in 23.05.
If there is a policy not to proceed with merge given 24.10 is branched already, please close.

@Neustradamus
Copy link

I hope that it will be possible to add the latest possibility in OpenWrt 23.05.x.
The World needs it.

@Neustradamus
Copy link

@hauke, @hnyman: What do you think?

@hnyman
Copy link
Contributor

hnyman commented Dec 9, 2024

for that to happen I will also need to PR a change to openwrt (not packages) so kmod-appletalk is also included in 23.05. I will submit that PR once this one is merged.

the kmod should be merged first to the main OpenWrt repo, I think, and this PR should depend on that getting accepted.

@APCCV
Copy link
Contributor Author

APCCV commented Dec 9, 2024

merged first to the main OpenWrt repo

Hey, @hnyman, thank you for taking a look. Snapshot and 24.10 are both up to date.

Snapshot - module: 5eb25dddb17fa3cf4958e91dfc9fc868c9eb03ac
Snapshot - package: 07efd50

24.10 - module: b2f3713
24.10 - package: 6952117

All tested and working on two targets, ipq806x and ath79. (Well... again, being transparent, DSA enabled caused a separate issue for which I have a patch ready but that is not applicable to 23.05).

The netatalk package can stand alone for AFP support. AFP (main feature in this package) doesn't require the module. AppleTalk (feature now available in v4) does. There is much value in merging this PR even without the kmod.

I will submit the PR for backport of the module and cross-reference.

@APCCV
Copy link
Contributor Author

APCCV commented Dec 9, 2024

PR for appletalk kmod sent... but (apologies for repeating) that is a very much "nice to have" for this package. Netatalk v4 will deliver functioning Apple File Sharing Protocol (AFPD) without the kmod.
With the kmod it will also deliver a functioning AppleTalk (atalkd).

@APCCV APCCV changed the title [23.05] netatalk: Update to 4.0.4 (backport) [23.05] netatalk: Update to 4.0.7 (backport) Dec 9, 2024
APCCV and others added 7 commits December 12, 2024 12:56
Commit restores package after it was removed from OpenWrt 21.02.
Signed-off-by: Antonio Pastor <apccv@outlook.com>
Compile tested/run: ipq806x

No changes to package other than using latest available codebase.

Signed-off-by: Antonio Pastor <apccv@outlook.com>
Maintainer: Antonio Pastor / @APCCV
Compile tested: ipq806x (23.05.3, snapshot)
Run tested: ipq806x - C2600: start server, connect from MacOS, read/write files to home share, create/update TimeMachine backups

Description:
No changes to package other than using latest available upstream code base. Starting Netatalk 4.x build uses meson instead of autotools.

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>
Updated title.
Updated description.
Reordered to match most common layout

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>
Starting v4.0.0, netatalk brings back AppleTalk features lost in v3.
Updated description.

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>
afpstats requires several libraries that would increase significantly final install size.

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>
Maintainer: Antonio Pastor / @APCCV
Compile tested: ipq806x (24.10-rc2, snapshot)
Run tested: ipq806x - C2600 (snapshot): install apk, start/stop sfpd service & review logs

Description:
Bump to version 4.0.7

Signed-off-by: Antonio Pastor <antonio.pastor@gmail.com>
@hnyman hnyman merged commit 2da11f9 into openwrt:openwrt-23.05 Dec 15, 2024
10 of 12 checks passed
@APCCV APCCV deleted the netatalk branch December 15, 2024 13:52
@Neustradamus
Copy link

@APCCV: Good job, thanks!
Thanks to @hnyman for merging :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants