Bug 4995 - Fix odd newline in $ARCH
Status: RESOLVED FIXED
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Platform
Version: GIT MASTER
Hardware: SGI IRIX
: P3 trivial
Assignee: Zachary J. Slater
QA Contact: ioquake3 bugzilla mailing list
URL:
Depends on:
Blocks:
 
Reported: 2011-05-21 16:31 EDT by Patrick Baggett
Modified: 2011-06-06 11:03:40 EDT
1 user (show)

See Also:


Attachments
Fixes the \n in the $ARCH variable for IRIX/mips (270 bytes, patch)
2011-05-21 16:31 EDT, Patrick Baggett

Description Patrick Baggett 2011-05-21 16:31:51 EDT
Created attachment 2719 [details]
Fixes the \n in the $ARCH variable for IRIX/mips

Using the default makefile, I get this:

Makefile:1870: warning: overriding commands for target `.so'
Makefile:1826: warning: ignoring old commands for target `.so'
Makefile:1923: warning: overriding commands for target `.so'
Makefile:1870: warning: ignoring old commands for target `.so'
Makefile:1974: warning: overriding commands for target `.so'
Makefile:1923: warning: ignoring old commands for target `.so'
Makefile:2037: warning: overriding commands for target `.so'
Makefile:1974: warning: ignoring old commands for target `.so'
Makefile:2065: warning: overriding commands for target `.so'
Makefile:2037: warning: ignoring old commands for target `.so'


[...a bit later...]

  Output:
    build/release-irix64-mips/ioq3ded.mips
    build/release-irix64-mips/ioquake3.mips
    build/release-irix64-mips/baseq3/cgamemips
    .so
    build/release-irix64-mips/baseq3/qagamemips
    .so
    build/release-irix64-mips/baseq3/uimips
    .so
    build/release-irix64-mips/missionpack/cgamemips
    .so
    build/release-irix64-mips/missionpack/qagamemips
    .so
    build/release-irix64-mips/missionpack/uimips
    .so


It looks like the "mips" is actually "mips\n" somehow. Patch to fix the makefile is attached, it basically removes extra whitespace around "ARCH=mips" line.
Comment 1 Thilo Schulz 2011-06-06 11:03:40 EDT
thank you, applied r2009