Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[bug]: Makefile downloader does not follow redirects. #136

Closed
1 task done
miguelangel-nubla opened this issue Jun 1, 2023 · 4 comments · Fixed by #137
Closed
1 task done

[bug]: Makefile downloader does not follow redirects. #136

miguelangel-nubla opened this issue Jun 1, 2023 · 4 comments · Fixed by #137
Assignees
Labels
bug support-level-core Functionality with core support level

Comments

@miguelangel-nubla
Copy link
Contributor

Severity

Blocker

Versions

Components/Modules

sounds

Operating Environment

Frequency of Occurrence

Constant

Issue Description

The wget run by the Makefile on building does not follow redirects.

Seems that a recent change to the host of http://downloads.asterisk.org/pub/telephony/sounds/releases/asterisk-core-sounds-en-ulaw-1.6.1.tar.gz triggered this.

Relevant log output

root@3d8a862be362:/usr/src/asterisk# make install
CC="cc" CXX="g++" LD="" AR="" RANLIB="" CFLAGS="" LDFLAGS="" make -C menuselect CONFIGURE_SILENT="--silent" makeopts
make[1]: Entering directory '/usr/src/asterisk/menuselect'
make[1]: 'makeopts' is up to date.
make[1]: Leaving directory '/usr/src/asterisk/menuselect'
Installing modules from channels...
Installing modules from pbx...
Installing modules from apps...
Installing modules from codecs...
Installing modules from formats...
Installing modules from cdr...
Installing modules from cel...
Installing modules from bridges...
Installing modules from funcs...
Installing modules from tests...
Installing modules from main...
Installing modules from res...
Installing modules from addons...
/usr/bin/install -c -m 755 contrib/scripts/astversion "/usr/sbin/"
/usr/bin/install -c -m 755 contrib/scripts/astgenkey "/usr/sbin/"
/usr/bin/install -c -m 755 contrib/scripts/autosupport "/usr/sbin/"
./build_tools/install_subst contrib/scripts/safe_asterisk "/usr/sbin/safe_asterisk";
/usr/bin/install -c -m 644 doc/core-*.xml "/var/lib/asterisk/documentation"
/usr/bin/install -c -m 644 doc/appdocsxml.xslt "/var/lib/asterisk/documentation"
/usr/bin/install -c -m 644 doc/appdocsxml.dtd "/var/lib/asterisk/documentation"
/usr/bin/install -c -m 644 doc/asterisk.8 "/usr/share/man/man8"
/usr/bin/install -c -m 644 doc/astdb*.8 "/usr/share/man/man8"
/usr/bin/install -c -m 644 contrib/scripts/astgenkey.8 "/usr/share/man/man8"
/usr/bin/install -c -m 644 contrib/scripts/autosupport.8 "/usr/share/man/man8"
/usr/bin/install -c -m 644 contrib/scripts/safe_asterisk.8 "/usr/share/man/man8"
if [ -f contrib/firmware/iax/iaxy.bin ] ; then \
        /usr/bin/install -c -m 644 contrib/firmware/iax/iaxy.bin "/var/lib/asterisk/firmware/iax/iaxy.bin"; \
fi
CFLAGS="  -I/usr/include/libxml2 -pipe -Wall -Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations   -g3 " build_tools/mkpkgconfig "/usr/lib/pkgconfig";
find static-http | while read x; do \
        if test -d $x; then \
                /usr/bin/install -c -m 755 -d "/var/lib/asterisk/$x"; \
        else \
                /usr/bin/install -c -m 644 $x "/var/lib/asterisk/$x" ; \
        fi \
done
/usr/bin/install -c -m 644 doc/core-en_US.xml "/var/lib/asterisk/static-http";
/usr/bin/install -c -m 644 doc/appdocsxml.xslt "/var/lib/asterisk/static-http";
if [ -d doc/tex/asterisk ] ; then \
        /usr/bin/install -c -d "/var/lib/asterisk/static-http/docs" ; \
        for n in doc/tex/asterisk/* ; do \
                /usr/bin/install -c -m 644 $n "/var/lib/asterisk/static-http/docs" ; \
        done \
fi
for x in images/*.jpg; do \
        /usr/bin/install -c -m 644 $x "/var/lib/asterisk/images" ; \
done
make -C sounds install
make[1]: Entering directory '/usr/src/asterisk/sounds'
#################################################################################################################################################################################################### 100.0%
http://downloads.asterisk.org/pub/telephony/sounds/releases/asterisk-core-sounds-en-ulaw-1.6.1.tar.gz

gzip: stdin: not in gzip format
tar: This does not look like a tar archive
tar: Exiting with failure status due to previous errors
make[1]: *** [Makefile:142: .asterisk-core-sounds-en-ulaw-1.6.1] Error 2
make[1]: Leaving directory '/usr/src/asterisk/sounds'
make: *** [Makefile:465: datafiles] Error 2

Asterisk Issue Guidelines

  • Yes, I have read the Asterisk Issue Guidelines
@jcolp
Copy link
Member

jcolp commented Jun 1, 2023

What version of wget is in use out of curiosity? Mine already follows redirects.

@jcolp
Copy link
Member

jcolp commented Jun 1, 2023

Oh, or are you using curl?

@miguelangel-nubla
Copy link
Contributor Author

Yes curl is the issue.

miguelangel-nubla added a commit to miguelangel-nubla/asterisk that referenced this issue Jun 1, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: asterisk#136
miguelangel-nubla added a commit to miguelangel-nubla/asterisk that referenced this issue Jun 1, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: asterisk#136
seanbright pushed a commit to miguelangel-nubla/asterisk that referenced this issue Jun 1, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: asterisk#136
@jcolp
Copy link
Member

jcolp commented Jun 5, 2023

The https redirect is now disabled.

gtjoseph pushed a commit that referenced this issue Jun 5, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
asteriskteam pushed a commit that referenced this issue Jun 5, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
asteriskteam pushed a commit that referenced this issue Jun 5, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
gtjoseph pushed a commit that referenced this issue Jun 5, 2023
asteriskteam pushed a commit that referenced this issue Jun 5, 2023
asteriskteam pushed a commit that referenced this issue Jun 5, 2023
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
(cherry picked from commit 4c2f035)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
Related to #136

(cherry picked from commit ade23d2)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
(cherry picked from commit 4c2f035)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
Related to #136

(cherry picked from commit ade23d2)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
(cherry picked from commit 4c2f035)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
Related to #136

(cherry picked from commit ade23d2)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
(cherry picked from commit 4c2f035)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
Related to #136

(cherry picked from commit ade23d2)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
(cherry picked from commit 4c2f035)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
Related to #136

(cherry picked from commit ade23d2)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
(cherry picked from commit 95d339a)
asterisk-org-access-app bot pushed a commit that referenced this issue Jun 30, 2023
Related to #136

(cherry picked from commit ada3dc2)
asterisk-org-access-app bot pushed a commit that referenced this issue Jul 10, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
(cherry picked from commit 4c2f035)
asterisk-org-access-app bot pushed a commit that referenced this issue Jul 10, 2023
Related to #136

(cherry picked from commit ade23d2)
asterisk-org-access-app bot pushed a commit that referenced this issue Jul 10, 2023
If curl is used for building, any download such as a sounds package
will fail to follow HTTP redirects and will download wrong data.

Resolves: #136
(cherry picked from commit 95d339a)
asterisk-org-access-app bot pushed a commit that referenced this issue Jul 10, 2023
Related to #136

(cherry picked from commit ada3dc2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug support-level-core Functionality with core support level
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants