[Differential] [Accepted] D3175: ar: add -U (unique) option to disable -D (deterministic) mode

2015-07-23 Thread imp (Warner Losh)
imp accepted this revision. imp added a reviewer: imp. This revision has a positive review. REVISION DETAIL https://reviews.freebsd.org/D3175 EMAIL PREFERENCES https://reviews.freebsd.org/settings/panel/emailpreferences/ To: emaste, brooks, bapt, imp Cc: freebsd-toolchain-list

[Differential] [Commented On] D1932: Remove the non-standard CC alias for c++

2015-05-26 Thread imp (Warner Losh)
imp added a comment. CC has been an alias for the default C++ compiler the entire life FreeBSD has had a C++ compiler. Sure, it isn't based on any standards but it is a very de-facto standard that many ports rely on. FreeBSD isn't the only system to do this. Sun did this for years, as did SGI,

[Differential] [Requested Changes To] D1932: Remove the non-standard CC alias for c++

2015-05-26 Thread imp (Warner Losh)
imp requested changes to this revision. imp added a reviewer: imp. imp added a comment. This revision now requires changes to proceed. So it is icky likely isn't a good enough reason. Do a ports run w/o it causing any problems and prove me wrong... REPOSITORY rS FreeBSD src repository

[Differential] [Accepted] D2576: Update crunch bootstrapping test

2015-05-18 Thread imp (Warner Losh)
imp accepted this revision. imp added a comment. This revision has a positive review. matches my memory, but I didn't verify the R numbers as correct. I assume Ed did, so I'm good with this in... REVISION DETAIL https://reviews.freebsd.org/D2576 EMAIL PREFERENCES

[Differential] [Accepted] D2408: Add ELF Tool Chain's c++filt to the build

2015-04-30 Thread imp (Warner Losh)
imp added a subscriber: imp. imp accepted this revision. imp added a reviewer: imp. imp added a comment. I came to troll you for TOLO - YOLO, but brooks beat me to it... REVISION DETAIL https://reviews.freebsd.org/D2408 EMAIL PREFERENCES

[Differential] [Updated] D2305: Fix bootstraping of crunchide

2015-04-17 Thread imp (Warner Losh)
imp added a comment. I think that we've stumbled on more stuff we can just kill :) INLINE COMMENTS Makefile.inc1:1469 I'd be inclined to promote it. Makefile.inc1:1470 Can you tell me where in the tree we define RELEASEDIR? Is it still even a thing? Makefile.inc1:1475 I'd love to know

[Differential] [Commented On] D2305: Fix bootstraping of crunchide

2015-04-17 Thread imp (Warner Losh)
imp added inline comments. INLINE COMMENTS Makefile.inc1:1476 kgzip is actually old, detritus, so I removed it in D2313. REVISION DETAIL https://reviews.freebsd.org/D2305 To: rodrigc, imp, emaste Cc: freebsd-toolchain ___

[Differential] [Commented On] D2305: Fix bootstraping of crunchide

2015-04-17 Thread imp (Warner Losh)
imp added inline comments. INLINE COMMENTS Makefile.inc1:1470 RELEASEDIR is detritus, so I removed it in D2313. It hasn't been a thing since FreeBSD 9's rewrite of the release process. REVISION DETAIL https://reviews.freebsd.org/D2305 To: rodrigc, imp, emaste Cc: freebsd-toolchain

[Differential] [Accepted] D2156: Switch to ELF toolchain readelf

2015-03-27 Thread imp (Warner Losh)
imp accepted this revision. imp added a comment. This revision is now accepted and ready to land. Looks good REVISION DETAIL https://reviews.freebsd.org/D2156 To: emaste, imp Cc: freebsd-toolchain ___ freebsd-toolchain@freebsd.org mailing list

[Differential] [Accepted] D2003: Do not strip crunched binary; it will be done by install

2015-03-03 Thread imp (Warner Losh)
imp accepted this revision. imp added a comment. This revision is now accepted and ready to land. Looks good to me. REVISION DETAIL https://reviews.freebsd.org/D2003 To: emaste, andrew, imp Cc: freebsd-toolchain ___ freebsd-toolchain@freebsd.org

[Differential] [Accepted] D1819: libdwarf: Handle .rel relocations

2015-02-11 Thread imp (Warner Losh)
imp accepted this revision. imp added a reviewer: imp. REVISION DETAIL https://reviews.freebsd.org/D1819 To: emaste, br, rpaulo, gnn, imp Cc: rpaulo, freebsd-toolchain ___ freebsd-toolchain@freebsd.org mailing list

[Differential] [Accepted] D1722: Add -s option to config

2015-01-29 Thread imp (Warner Losh)
imp accepted this revision. imp added a comment. This revision is now accepted and ready to land. Looks good to me. REVISION DETAIL https://reviews.freebsd.org/D1722 To: rodrigc, brooks, imp Cc: freebsd-toolchain ___ freebsd-toolchain@freebsd.org

[Differential] [Accepted] D1682: Preserve hard symbolic links when modifying source file

2015-01-28 Thread imp (Warner Losh)
imp accepted this revision. imp added a reviewer: imp. REVISION DETAIL https://reviews.freebsd.org/D1682 To: emaste, rpaulo, imp Cc: freebsd-toolchain ___ freebsd-toolchain@freebsd.org mailing list

[Differential] [Accepted] D1543: crunchide: Correct 64-bit section header offset

2015-01-16 Thread imp (Warner Losh)
imp accepted this revision. imp added a reviewer: imp. This revision is now accepted and ready to land. REVISION DETAIL https://reviews.freebsd.org/D1543 To: emaste, pfg, imp Cc: freebsd-toolchain ___ freebsd-toolchain@freebsd.org mailing list

[Differential] [Accepted] D1468: Fix the ARM build of compiler-rt

2015-01-09 Thread imp (Warner Losh)
imp accepted this revision. imp added a reviewer: imp. This revision is now accepted and ready to land. BRANCH /head REVISION DETAIL https://reviews.freebsd.org/D1468 To: andrew, loos, sbruno, rpaulo, ian, dim, imp Cc: freebsd-toolchain ___

[Differential] [Accepted] D1327: Do not strip all when stripping an explicit symbol

2014-12-16 Thread imp (Warner Losh)
imp added a subscriber: imp. imp accepted this revision. imp added a reviewer: imp. imp added a comment. This revision is now accepted and ready to land. Looks good to me. REVISION DETAIL https://reviews.freebsd.org/D1327 To: emaste, imp Cc: imp, freebsd-toolchain