Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: turn on thin static archives #7957

Merged
merged 1 commit into from
Aug 9, 2016

Commits on Aug 9, 2016

  1. build: turn on thin static archives

    Thin archives were disabled in 2012 as a workaround (IIRC) for obsolete
    tooling on one of Joyent's platforms.  The last binutils versions that
    didn't support them was released in 2007 so I think it's safe to assume
    we can drop support for that now - except on SmartOS, where the tooling
    still has a distinctive vintage feel to it.
    
    Thin archives save space - it shrinks the size of PRODUCT_DIR by 30% -
    and speed up the final linking step because it doesn't have to assemble
    50 MB of static archives (twice! - first to create the archive, then to
    copy it to PRODUCT_DIR).  The archives are just 3.5 MB now and no longer
    copied around.
    
    PR-URL: nodejs#7957
    Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    bnoordhuis committed Aug 9, 2016
    Configuration menu
    Copy the full SHA
    e03a7b2 View commit details
    Browse the repository at this point in the history