Skip to content

Conversation

@jslachta
Copy link
Contributor

@jslachta jslachta commented Dec 27, 2025

Maintainer: me / jiri@slachta.eu
Compile tested: CI passed on all targets

Description:

This PR removes the app-voicemail-imap module from Asterisk.

The module depends on uw-imap, which has been removed from OpenWrt packages feed as it is
unmaintained and no longer available:
openwrt/packages#28113

There is currently no alternative IMAP implementation in OpenWrt that this
module could depend on, so the module is removed.

Fixes #928

@systemcrash
Copy link
Contributor

systemcrash commented Dec 28, 2025

Would be great to document the reason for the removal: its dependent module was removed, and there is no available substitute, so it broke compile. Perhaps a link to the PR that removed the uw-imap tool. Then just merge, if test builds complete.

@BKPepe
Copy link
Member

BKPepe commented Dec 28, 2025

there is no available substitute

There is according to https://docs.asterisk.org/Configuration/Applications/Voicemail/IMAP-Voicemail-Storage/IMAP-Server-Implementations/ , it might be possible to use it with Cyrus imap and someone should raise issue with upstream about uw-imap.

@systemcrash
Copy link
Contributor

Please merge.

@BKPepe
Copy link
Member

BKPepe commented Jan 3, 2026

Draft PR can not be merged:-)

@jslachta jslachta changed the title asterisk: remove app-voicemail-imap module [25.12] asterisk: remove app-voicemail-imap module Jan 3, 2026
Removed app-voicemail-imap from the Asterisk modules.

The module depends on uw-imap, which has been removed from OpenWrt packages feed as it is unmaintained and no longer available.

There is currently no alternative IMAP implementation in OpenWrt that this module could depend on, so the module is removed.

Signed-off-by: Jiri Slachta <jiri@slachta.eu>
@jslachta jslachta marked this pull request as ready for review January 3, 2026 21:14
@jslachta
Copy link
Contributor Author

jslachta commented Jan 3, 2026

Apologies for the delay, my kids are sick. The description has been updated and the commit message amended. Waiting for all green checks, but we can merge immediately. I changed the PR to not be a draft.

@BKPepe
Copy link
Member

BKPepe commented Jan 3, 2026

Shouldnt this go to the master branch first?

@jslachta
Copy link
Contributor Author

jslachta commented Jan 3, 2026

It depends. This PR follows a standard hotfix workflow: fix the affected release branch first, then forward-port it to master.

@jslachta jslachta merged commit 2618106 into openwrt:openwrt-25.12 Jan 4, 2026
11 checks 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.

3 participants