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

bashism in configure script #113

Closed
Zugschlus opened this issue Nov 7, 2021 · 1 comment
Closed

bashism in configure script #113

Zugschlus opened this issue Nov 7, 2021 · 1 comment
Labels
bug Something isn't working
Milestone

Comments

@Zugschlus
Copy link

This is Debian Bug #998766, filed by Andrej Shadura

Your package uses configure script with bash features not present in
POSIX without explicitly declaring the need to bash shell; this
currently works as configure scripts select bash, but when dash enables
LINENO support, your configure script will start failing:

 ../configure: 7171: test: yes: unexpected operator                                       
 checking for pcre_exec in -lpcre... yes                                                  
 checking for posix-acl-support... yes                                                    
 checking for selinux-support... yes                                                      
 ../configure: 7382: test: yes: unexpected operator                                       
 checking for prelink-support... no                                                       
 checking for xattr-support... yes                                                        
 checking for capabilities-support... yes                                                 
 checking for e2fsattrs-support... yes                                                    
 checking for pkg-config... (cached) /usr/bin/pkg-config                                  
 checking pkg-config is at least version 0.9.0... yes                                     
 checking for check >= 0.9.4... no                                                        
 Check not found (testing via 'make check' disabled)                                      
 ../configure: 7848: test: no: unexpected operator                                        
 checking for zlib.h... yes                                                               
 checking for deflate in -lz... yes                                                       
 checking for mhash_get_block_size in -lmhash... yes                                      
 checking for libaudit.h... yes                                                           
 ../configure: 8354: test: yes: unexpected operator                                       
 checking that generated files are newer than configure... done                           

To test this, you can install dash from experimental and re-run the
configure script.

Please replace non-POSIX features with their equivalents to make sure
the script runs with dash. Most common ones are usage of == instead of =
and for with arrays (not lists).

@hvhaugwitz hvhaugwitz added the bug Something isn't working label Nov 7, 2021
@hvhaugwitz hvhaugwitz added this to the 0.18 milestone Nov 7, 2021
@hvhaugwitz
Copy link
Member

fixed in 007eb4f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants