WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
123971
Jhbuild build bug workaround
https://bugs.webkit.org/show_bug.cgi?id=123971
Summary
Jhbuild build bug workaround
Przemyslaw Szymanski
Reported
2013-11-07 00:46:54 PST
Jhbuild build uses Makefile.plain lcated in po directory (WebKitBuild/Dependencies/Source/jhbuild/po). This makefile file uses grep -v '^\#' LINGUAS Output from this grep is: ca cs de el en_GB (...) If someone has set grep env variable GREP_OPTIONS=-n (or other that changes grep output) then this output will be: 2:ca 3:cs 4:de 5:el 6:en_GB (...) The line is added by grep. After that jhbuild will not build because this grep output is different than expected. If you will try to build jhbuild (make install) with this grep option output will be: make[1]: Entering directory `WebKit/WebKitBuild/Dependencies/Source/jhbuild/po' Makefile.plain:10: *** target pattern contains no `%'. Stop. make[1]: Leaving directory `WebKit/WebKitBuild/Dependencies/Source/jhbuild/po' make: *** [install] Error 2 This patch is a WebKit workaround to this jhbuild issue. It doesn't changes anything else with WebKit build.
Attachments
jhbuild fix
(1.64 KB, patch)
2013-11-07 00:49 PST
,
Przemyslaw Szymanski
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Przemyslaw Szymanski
Comment 1
2013-11-07 00:49:10 PST
Created
attachment 216270
[details]
jhbuild fix
Ryosuke Niwa
Comment 2
2013-11-07 01:00:35 PST
Comment on
attachment 216270
[details]
jhbuild fix rs=me.
Przemyslaw Szymanski
Comment 3
2013-11-08 01:15:49 PST
(In reply to
comment #2
)
> (From update of
attachment 216270
[details]
) > rs=me.
Thank you for review. cq?
WebKit Commit Bot
Comment 4
2013-11-08 01:41:44 PST
Comment on
attachment 216270
[details]
jhbuild fix Clearing flags on attachment: 216270 Committed
r158912
: <
http://trac.webkit.org/changeset/158912
>
WebKit Commit Bot
Comment 5
2013-11-08 01:41:46 PST
All reviewed patches have been landed. Closing bug.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug