-
Notifications
You must be signed in to change notification settings - Fork 42
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
Make make distcheck
pass
#159
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Just one comment regarding limits.
@@ -3,7 +3,7 @@ AC_INIT([pick], [1.4.0], [[email protected]]) | |||
AM_INIT_AUTOMAKE([subdir-objects]) | |||
AC_CONFIG_HEADERS([config.h]) | |||
AC_PROG_CC | |||
AC_CHECK_HEADERS([locale.h stdlib.h string.h unistd.h]) | |||
AC_CHECK_HEADERS([limits.h locale.h stdlib.h string.h unistd.h]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The sys/limits.h -> limits.h
change makes sense and is more
portable. However, I would not bother adding the extra check since
limits.h is defined by POSIX[1].
[1] http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/limits.h.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great point! Fixed in 64a5107.
@@ -3,6 +3,9 @@ | |||
/* Define to 1 if you have the <inttypes.h> header file. */ | |||
#undef HAVE_INTTYPES_H | |||
|
|||
/* Define to 1 if you have the <limits.h> header file. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See comment below.
Thanks for the feedback @mptre! I'll merge this. |
`<sys/limits.h>` isn't available on macOS.
`make distcheck` was failing on the `check` task during the VPATH build due to two reasons: 1. The `tests/*.t` and `tests/test.sh` were missing in the distribution. 2. The `T_LOG_COMPILER` command could not find the file `tests/test.sh` since it was located in the source directory, not the build directory.
64a5107
to
b3c5bb1
Compare
Preparing for release, I ran
make distcheck
and found it failing. With the changes in this PR it is passing.@mptre, I'd love your feedback on the changes and also if you could try running it on your system.