Closed Bug 496860 Opened 12 years ago Closed 12 years ago
sed: command garbled for SOURCE
The patch should be applied for mail, suite, calender. +++ This bug was initially created as a clone of Bug #479762 +++ In browser/app, run make on Solaris 10 Got sed: command garbled error in terminal. I found similar code in toolkit/xre/Makefile.in and toolkit/content/Makefile.in. Should we copy it to browser/app/Makefile.in? +# strip a trailing slash from the repo URL because it's not always present, +# and we want to construct a working URL in buildconfig.html +# make+shell+sed = awful +_dollar=$$ +SOURCE_REPO := $(shell cd $(topsrcdir) && hg showconfig paths.default 2>/dev/null | head -n1 | sed -e "s/^ssh:/http:/" -e "s/\/$(_dollar)//" ) +
Assignee: nobody → ginn.chen
Status: NEW → ASSIGNED
Attachment #382098 - Flags: review?(ted.mielczarek)
FYI we normally put "comm-central" build config bugs under mailnews core (even though that isn't as "global" as it could be.
Product: Thunderbird → MailNews Core
QA Contact: build-config → build-config
Comment on attachment 382098 [details] [diff] [review] patch I'm sure Ted would be happy to review this, but no need to increase his load with comm-central changes. r=Standard8
Attachment #382098 - Flags: review?(ted.mielczarek) → review+
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.