other-licenses/snappy/src/snappy-c.h:49: error: comma at end of enumerator list

RESOLVED FIXED in mozilla15

Status

()

Core
Build Config
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: gaston, Assigned: gaston)

Tracking

Trunk
mozilla15
Other
OpenBSD
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 624329 [details] [diff] [review]
Remove extra comma at end of enum list

snappy wasn't built previously, dunno what changed recently that made that error surface.
(Assignee)

Updated

5 years ago
Attachment #624329 - Attachment is patch: true
Attachment #624329 - Flags: review?(Ms2ger)
Could you submit your patch to <http://code.google.com/p/snappy/> first? I think this is currently a straight import, so I'd like to avoid forking...

Updated

5 years ago
Assignee: nobody → landry
(Assignee)

Comment 2

5 years ago
http://code.google.com/p/snappy/issues/detail?id=62
Comment on attachment 624329 [details] [diff] [review]
Remove extra comma at end of enum list

Review of attachment 624329 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks. I'm fine with landing this on m-c before it's fixed upstream, if you keep track of the issue there.
Attachment #624329 - Flags: review?(Ms2ger) → review+
(Assignee)

Comment 4

5 years ago
Looking deeply into that..

in my previous builds, snappy-c.cc wasnt built/compiled, but was passed to expandlibs_gen.py :

/usr/local/bin/python2.7 /var/buildslave-mozilla/mozilla-central-amd64/build/config/pythonpath.py -I../../config /var/buildslave-mozilla/mozilla-central-amd64/build/config/expandlibs_gen.py snappy-c.cc snappy.o snappy-sinksource.o snappy-stubs-internal.o   > libsnappy_s.a.desc

In the failing build, it is, but with g++ instead of gcc (as snappy-c.cc is in CSRCS) So there might be two different issues here. Something changed that triggered snappy-c.cc to be compiled, and without the correct compiler it seems.
(Assignee)

Comment 5

5 years ago
http://buildbot.rhaalovely.net/builders/mozilla-central-amd64/builds/388/steps/build/logs/stdio for the failing build log
http://buildbot.rhaalovely.net/builders/mozilla-central-amd64/builds/386/steps/build/logs/stdio for a non-failing build log (look for snappy-c)

So something changed in the past two days that triggered snappy-c.cc to be built.
(Assignee)

Comment 6

5 years ago
Seems it's now in CPPSRCS, since bug 748001 / https://hg.mozilla.org/mozilla-central/rev/43cc6c43f79c - and that's what triggered the new build failure.
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/0f7d47f6b6dd
Flags: in-testsuite-
Keywords: checkin-needed
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/0f7d47f6b6dd
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.