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

delete stray incorrect Unicode characters #312

Merged
merged 1 commit into from
Mar 3, 2024

Conversation

timoteostewart
Copy link
Contributor

@timoteostewart timoteostewart commented Mar 3, 2024

Prerequisites

  • I have written a descriptive pull-request title
  • I have verified that there are no overlapping pull-requests open
  • I have verified that I am following the existing coding patterns and practices as demonstrated in the repository.

Description

See below for output of running configure. I saw the unusual command not found errors. I looked at the lines indicated and discovered some odd Unicode characters. I replaced the octal characters \357\277\274 (in hex: \xEF\xBF\xBC), with spaces in my local copy of configure, and ran it again. This seemed to resolve the problem.

From my limited exploration of where these characters came from, it looks like they were introduced in commit 119eb49 into lines 31599 and 31600 (as they were back then).

@urban-warrior

-------------------------------------------------------------
checking for JBIG...
checking for jbig.h... yes
checking for jbg_dec_init in -ljbig... yes
checking if JBIG package is complete... yes
-------------------------------------------------------------
checking for JPEG...
checking for libjpeg >= 2.0.0... yes
./configure: line 31540: $'\357\277\274': command not found
./configure: line 31541: $'\357\277\274': command not found
checking for jconfig.h... yes
checking for jerror.h... yes
checking for jmorecfg.h... yes
checking for jpeglib.h... yes
checking for jpeg_read_header in -ljpeg... yes
checking if JPEG package is complete... yes
-------------------------------------------------------------
checking for lcms2 >= 2.0.0... yes

checking for lcms2/lcms2.h... no
-------------------------------------------------------------

@urban-warrior urban-warrior merged commit 1f19f57 into ImageMagick:main Mar 3, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants