| Summary: | 2.1_rc1-r2 prepstrip gets path wrong for split-debug files | ||
|---|---|---|---|
| Product: | Portage Development | Reporter: | Kevin F. Quinn (RETIRED) <kevquinn> |
| Component: | Core | Assignee: | Portage team <dev-portage> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | flameeyes, marc |
| Priority: | High | Keywords: | InVCS, REGRESSION |
| Version: | 2.1 | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
| Bug Depends on: | |||
| Bug Blocks: | 115839 | ||
| Attachments: | Remove '+1' from ${x:${#D}+1}, and -print0 => -print so the bash for loop works. | ||
Created attachment 86935 [details, diff]
Remove '+1' from ${x:${#D}+1}, and -print0 => -print so the bash for loop works.
(the -print0 causes bash to process the find output as one big string, as it doesn't consider '\0' as a separator)
Thanks, this is in svn r3373. *** Bug 133693 has been marked as a duplicate of this bug. *** This has been released in 2.1_rc1-r3. *** Bug 133719 has been marked as a duplicate of this bug. *** |
In prepstrip, save_elf_debug() we have: local y="${D}usr/lib/debug/${x:${#D}+1}.debug" which is causing the first character after ${D} to be removed. I think the '+1' just needs to be removed (here and on two vecho lines further on). Also there's a '-print0' on a find command, which I think should be just '-print'. Actually - I'll attach a diff to be clear :)