failing to build 'joe' #13818

Closed
isotopp opened this Issue Jul 31, 2012 · 3 comments

Projects

None yet

3 participants

@isotopp
isotopp commented Jul 31, 2012

I am trying 'brew install joe' for the editor of that name. It dies with a charset problem in sed.

Making install in ru
sed 's,@JOERC@,/usr/local/Cellar/joe/3.7/etc/joe,; s,@JOEDOC@,/usr/local/Cellar/joe/3.7/share/doc/joe,; s,@JOEDATA@,/usr/local/Cellar/joe/3.7/share/joe,; s,@SPELL@,ispell,; s,@BINDIR@,/usr/local/Cellar/joe/3.7/bin,' ./joe.1.in > joe.1
sed: RE error: illegal byte sequence
make[2]: *** [joe.1] Error 1
make[1]: *** [install-recursive] Error 1
make: *** [install-recursive] Error 1
config.log was copied to /Users/kris/Library/Logs/Homebrew

brew --config and brew install -v joe output: https://gist.github.com/3216401
configure output: https://gist.github.com/3216406

@mxcl
Member
mxcl commented Aug 2, 2012

Confirmed that it's the LANG issue.

@MikeMcQuaid MikeMcQuaid added a commit that closed this issue Aug 2, 2012
@MikeMcQuaid MikeMcQuaid Fix Mountain Lion sed by setting LANG.
It's also necessary to delete LC_ALL or it overrides the new LANG.
LC_COLLATE is the problem variable but is affected by LANG and LC_ALL
so just use them instead. Extends the fix made to geoip in dc955c.

Fix #11728, fix #12890, fix #13653, fix #13734, fix #13787, fix #13818.
00ba4df
@MikeMcQuaid
Member

Pushed a fix in 00ba4df.

@Sharpie Sharpie pushed a commit to Sharpie/homebrew that referenced this issue Aug 2, 2012
@MikeMcQuaid MikeMcQuaid Fix Mountain Lion sed by setting LANG.
It's also necessary to delete LC_ALL or it overrides the new LANG.
LC_COLLATE is the problem variable but is affected by LANG and LC_ALL
so just use them instead. Extends the fix made to geoip in dc955c.

Fix #11728, fix #12890, fix #13653, fix #13734, fix #13787, fix #13818.
c24480d
@isotopp
isotopp commented Aug 6, 2012

On 02.08.2012, at 22:02, Mike McQuaid reply@reply.github.com wrote:

Pushed a fix in 00ba4df.

Works for me. After a brew update, I get

KK:~ kris$ brew install joe
==> Downloading http://downloads.sourceforge.net/project/joe-editor/JOE%20source
Already downloaded: /Library/Caches/Homebrew/joe-3.7.tar.gz
==> ./configure --prefix=/usr/local/Cellar/joe/3.7
==> make install
/usr/local/Cellar/joe/3.7: 69 files, 1,5M, built in 26 seconds

Kris

Kristian Köhntopp
https://www.xing.com/profile/Kristian_Koehntopp
http://blog.koehntopp.de

@Sharpie Sharpie pushed a commit to Sharpie/homebrew that referenced this issue Aug 13, 2012
@MikeMcQuaid MikeMcQuaid Use LC_CTYPE rather than LANG for most specific ML sed fix.
Fix #11728, fix #12890, fix #13653, fix #13734, fix #13787, fix #13818,
fix #13939.
00e5768
@fgeller fgeller added a commit to fgeller/homebrew that referenced this issue Aug 22, 2012
@MikeMcQuaid MikeMcQuaid Fix Mountain Lion sed by setting LANG.
It's also necessary to delete LC_ALL or it overrides the new LANG.
LC_COLLATE is the problem variable but is affected by LANG and LC_ALL
so just use them instead. Extends the fix made to geoip in dc955c.

Fix #11728, fix #12890, fix #13653, fix #13734, fix #13787, fix #13818.
a9045c5
@fgeller fgeller added a commit to fgeller/homebrew that referenced this issue Aug 22, 2012
@MikeMcQuaid MikeMcQuaid Use LC_CTYPE rather than LANG for most specific ML sed fix.
Fix #11728, fix #12890, fix #13653, fix #13734, fix #13787, fix #13818,
fix #13939.
4dede1b
@Sharpie Sharpie pushed a commit to Sharpie/homebrew that referenced this issue Sep 12, 2012
@MikeMcQuaid MikeMcQuaid Fix Mountain Lion sed by setting LANG.
It's also necessary to delete LC_ALL or it overrides the new LANG.
LC_COLLATE is the problem variable but is affected by LANG and LC_ALL
so just use them instead. Extends the fix made to geoip in dc955c.

Fix #11728, fix #12890, fix #13653, fix #13734, fix #13787, fix #13818.
736a75c
@snakeyroc3 snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this issue Dec 17, 2012
@MikeMcQuaid MikeMcQuaid Fix Mountain Lion sed by setting LANG.
It's also necessary to delete LC_ALL or it overrides the new LANG.
LC_COLLATE is the problem variable but is affected by LANG and LC_ALL
so just use them instead. Extends the fix made to geoip in dc955c.

Fix #11728, fix #12890, fix #13653, fix #13734, fix #13787, fix #13818.
0978e15
@snakeyroc3 snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this issue Dec 17, 2012
@MikeMcQuaid MikeMcQuaid Use LC_CTYPE rather than LANG for most specific ML sed fix.
Fix #11728, fix #12890, fix #13653, fix #13734, fix #13787, fix #13818,
fix #13939.
ba46d61
@xu-cheng xu-cheng locked and limited conversation to collaborators Feb 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.