Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(622)

Issue 14713006: Retain the unstripped nexe at a predictable location. (Closed)

Created:
7 years, 7 months ago by bradn
Modified:
7 years, 7 months ago
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

Retain the unstripped nexe at a predictable location. Retaining the unstripped nexe at a predictable location in order to allow us to archive a copy of the unstripped IRT, and to have it available locally. Add a gnu_debuglink link to the stripped irt (and other nexes/so) to facilitate the debugger finding the debugging info. Now that we have a process to retain debug info, strip fully for irt_core and tests. BUG=http://code.google.com/p/chromium/issues/detail?id=117242 TEST=builds R=jvoung@chromium.org Committed: https://src.chromium.org/viewvc/native_client?view=rev&revision=11290

Patch Set 1 #

Patch Set 2 : changed ext #

Patch Set 3 : fix bug #

Patch Set 4 : review fixes. #

Patch Set 5 : add more stripping #

Unified diffs Side-by-side diffs Delta from patch set Stats (+67 lines, -18 lines) Patch
M build/build_nexe.py View 1 2 3 7 chunks +63 lines, -17 lines 0 comments Download
M src/untrusted/irt/irt.gyp View 1 2 3 4 1 chunk +3 lines, -0 lines 0 comments Download
M tests.gyp View 1 2 3 4 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 10 (0 generated)
bradn
7 years, 7 months ago (2013-05-01 00:45:31 UTC) #1
bradn
7 years, 7 months ago (2013-05-01 00:45:31 UTC) #2
khimg
Flyby comment: why not create standard .dbg file linked to .nexe? http://sourceware.org/gdb/download/onlinedocs/gdb/Separate-Debug-Files.html#Separate-Debug-Files
7 years, 7 months ago (2013-05-01 01:09:55 UTC) #3
jvoung (off chromium)
lgtm
7 years, 7 months ago (2013-05-01 19:29:00 UTC) #4
Mark Seaborn
On 2013/05/01 01:09:55, khimg wrote: > Flyby comment: why not create standard .dbg file linked ...
7 years, 7 months ago (2013-05-01 20:33:20 UTC) #5
bradn
Ok, switched to .debug, and adding the gnu_debuglink. This would use the crc for now, ...
7 years, 7 months ago (2013-05-02 20:06:10 UTC) #6
bradn
Committed patchset #5 manually as r11290 (presubmit successful).
7 years, 7 months ago (2013-05-02 22:48:28 UTC) #7
khim
On Fri, May 3, 2013 at 12:06 AM, <bradnelson@google.com> wrote: > Ok, switched to .debug, ...
7 years, 7 months ago (2013-05-03 11:39:44 UTC) #8
bradn
Oops, didn't think thru that case. Sorry apparently or default trybot list doesn't cover the ...
7 years, 7 months ago (2013-05-03 15:45:22 UTC) #9
mseaborn - please use other
7 years, 7 months ago (2013-05-03 18:10:10 UTC) #10
On 3 May 2013 04:39, Victor Khimenko <khim@chromium.org> wrote:

> On Fri, May 3, 2013 at 12:06 AM, <bradnelson@google.com> wrote:
>
>> Ok, switched to .debug, and adding the gnu_debuglink.
>>
>
> Sadly this fails on nacl_sdk bots:
>
>
http://build.chromium.org/p/tryserver.chromium/builders/linux_nacl_sdk/builds...
>
> /bin/sh: 1:
../../../../../out/gypbuild-pnacl-arm/Release/gen/sdk/toolchain/linux_x86_pnacl/newlib/bin/pnacl-objcopy:
not found
>
>
> Probably the simplest fix is to disable it for pnacl ...
>

I get the same error when attempting to roll nacl_revision in Chromium's
DEPS:

chromeos-chrome-28.0.1497.1_rc-r1: /bin/sh:
/home/chrome-bot/chrome_root/src/c/Release/obj/gen/sdk/toolchain/linux_x86_pnacl/newlib/bin/pnacl-objcopy:
No such file or directory
chromeos-chrome-28.0.1497.1_rc-r1: Err 127: nacl-pnacl-objcopy
/home/chrome-bot/chrome_root/src/c/Release/obj/gen/tc_pnacl_translate/lib-arm/libpnacl_irt_shim.a
- see
http://build.chromium.org/p/tryserver.chromium/builders/cros_daisy/builds/170...
and https://codereview.chromium.org/14771013/

I'm closing the NaCl tree until this is verified as fixed.

Mark

-- 
You received this message because you are subscribed to the Google Groups
"Native-Client-Reviews" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to native-client-reviews+unsubscribe@googlegroups.com.
To post to this group, send email to native-client-reviews@googlegroups.com.
Visit this group at http://groups.google.com/group/native-client-reviews?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Powered by Google App Engine
This is Rietveld 408576698