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

Error while makeing project #338

Closed
dushmis opened this issue Mar 13, 2014 · 5 comments
Closed

Error while makeing project #338

dushmis opened this issue Mar 13, 2014 · 5 comments

Comments

@dushmis
Copy link

dushmis commented Mar 13, 2014

../dist/j2objcc -c -fobjc-abi-version=2 -fobjc-legacy-dispatch -I/opt/dushyant/j2objc/j2objc/guava/build_result/objc -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk -c /opt/dushyant/j2objc/j2objc/guava/build_result/objc/com/google/common/base/CaseFormat.m -o /opt/dushyant/j2objc/j2objc/guava/build_result/objs-macosx/com/google/common/base/CaseFormat.o
/opt/dushyant/j2objc/j2objc/guava/build_result/objc/com/google/common/base/CaseFormat.m:69:118: error: multiple unsequenced modifications to 'j' [-Werror,-Wunsequenced]
  while ((j = [((ComGoogleCommonBaseCharMatcher *) nil_chk(wordBoundary_)) indexInWithJavaLangCharSequence:s withInt:++j]) != -1) {
            ~                                                                                                        ^
1 error generated.
make[1]: *** [/opt/dushyant/j2objc/j2objc/guava/build_result/objs-macosx/com/google/common/base/CaseFormat.o] Error 1
make: *** [guava_dist] Error 2
make print_environment
Locale: en_US.UTF-8
Darwin mac 13.1.0 Darwin Kernel Version 13.1.0: Thu Jan 16 19:40:37 PST 2014; root:xnu-2422.90.20~2/RELEASE_X86_64 x86_64
Xcode 5.1 Build version 5B130a
Apple LLVM version 5.1 (clang-503.0.38) (based on LLVM 3.4svn)
Target: x86_64-apple-darwin13.1.0
Thread model: posix
@kstanger
Copy link
Collaborator

Are you sure that you're building from an unmodified client? When I build locally, that statement is rewritten to the following:

while (YES) {
int unseq$1 = ++j;
if (!((j = [((ComGoogleCommonBaseCharMatcher *) nil_chk(wordBoundary_)) indexInWithJavaLangCharSequence:s withInt:unseq$1]) != -1)) break;

Make sure that the guava files are translated using the --extract-unsequenced flag. (it should already be added in the guava makefile)

@dushmis
Copy link
Author

dushmis commented Mar 13, 2014

-Wno-unsequenced helped, and no i don't see --extract-unsequenced under guava Makefile

TRANSLATE_ARGS = --segmented-headers -cp $(CLASSPATH) -sourcepath $(SOURCEPATH) -encoding UTF-8

Weird, i just cloned the repo... although i see it in -- https://github.com/google/j2objc/blob/master/guava/Makefile#L38

@tomball
Copy link
Collaborator

tomball commented Mar 13, 2014

Try refreshing your cache of that page, as when I clicked your URL, that
line read: "TRANSLATE_ARGS = --segmented-headers --extract-unsequenced "

On Thu Mar 13 2014 at 9:14:15 AM, dushyant notifications@github.com wrote:

-Wno-unsequenced helped, and no i don't see --extract-unsequenced under
guava Makefile

https://github.com/google/j2objc/blob/master/guava/Makefile#L38


Reply to this email directly or view it on GitHubhttps://github.com//issues/338#issuecomment-37552272
.

@tomball
Copy link
Collaborator

tomball commented Mar 13, 2014

And do a "git pull" to make sure your repository is up-to-date.

On Thu Mar 13 2014 at 10:52:21 AM, Thomas Ball tball@google.com wrote:

Try refreshing your cache of that page, as when I clicked your URL, that
line read: "TRANSLATE_ARGS = --segmented-headers --extract-unsequenced "

On Thu Mar 13 2014 at 9:14:15 AM, dushyant notifications@github.com
wrote:

-Wno-unsequenced helped, and no i don't see --extract-unsequenced under
guava Makefile

https://github.com/google/j2objc/blob/master/guava/Makefile#L38


Reply to this email directly or view it on GitHubhttps://github.com//issues/338#issuecomment-37552272
.

@dushmis dushmis closed this as completed Mar 17, 2014
@dushmis dushmis reopened this Mar 17, 2014
@tomball
Copy link
Collaborator

tomball commented Jul 29, 2014

No further feedback, assume fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants