Skip to content

add build for chan_quectel - #343

Open
litinoveweedle wants to merge 2 commits into
TECH7Fox:mainfrom
litinoveweedle:chan_quectel
Open

add build for chan_quectel#343
litinoveweedle wants to merge 2 commits into
TECH7Fox:mainfrom
litinoveweedle:chan_quectel

Conversation

@litinoveweedle

Copy link
Copy Markdown

No description provided.

@TECH7Fox TECH7Fox self-assigned this Aug 14, 2024
@TECH7Fox TECH7Fox added the enhancement New feature or request label Aug 14, 2024
@TECH7Fox

Copy link
Copy Markdown
Owner

Thanks for the PR @litinoveweedle. How long have you been running this version? And does this module needs to manually be enabled? I'm worried about the long-term stability of this module.

Comment thread asterisk/Dockerfile
Comment on lines +243 to +244
# install headers
make install-headers

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# install headers
make install-headers
# install headers for building chan-quectel
make install-headers

Comment thread asterisk/Dockerfile
WORKDIR /usr/src/asterisk-chan-quectel
# https://github.com/RoEdAl/asterisk-chan-quectel/commits/master
ARG CHAN_QUECTEL_VERSION="5552c365bfb319eed7cbbf6300a67028ab70db9e"
RUN curl -fsSL "https://github.com/RoEdAl/asterisk-chan-quectel/archive/${CHAN_QUECTEL_VERSION}.tar.gz" | \

@felipecrs felipecrs Jul 20, 2025

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The https://github.com/RoEdAl/asterisk-chan-quectel is archived.

Have you seen https://github.com/IchthysMaranatha/asterisk-chan-quectel? It has been updated more recently, but maybe it lacks the features you need.

If https://github.com/RoEdAl/asterisk-chan-quectel is the best module even though archived, I think we can keep it at least while it doesn't break anything.

@felipecrs felipecrs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@litinoveweedle please also update asterisk/rootfs/etc/asterisk/modules.conf to avoid loading this module by default.

@felipecrs

Copy link
Copy Markdown
Collaborator

@litinoveweedle any plans to finish this PR?

@litinoveweedle

Copy link
Copy Markdown
Author

Hello I am currently travelling without access to PC. I can review this but earliest in 3weeks.

1 similar comment
@litinoveweedle

Copy link
Copy Markdown
Author

Hello I am currently travelling without access to PC. I can review this but earliest in 3weeks.

@felipecrs

Copy link
Copy Markdown
Collaborator

@litinoveweedle no problem and no rush at all. Just let us know in case you have no plans to finish the PR, though.

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

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants