Skip to content

doc: add ccache 4 windows for speeding up buiding section #52255

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.

Sign up for GitHub

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

Closed
congzhangzh wants to merge 10 commits into from
Closed

doc: add ccache 4 windows for speeding up buiding section #52255

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
7b17abd
doc: add ccache suport 4 windows
congzhangzh Mar 28, 2024
2d7668a
doc: minor fix
congzhangzh Mar 28, 2024
70a6f1a
Merge branch 'nodejs:main' into doc_buiding_speed_up_4_windows
congzhangzh Mar 30, 2024
512436f
doc: speed building on windows platform
congzhangzh Mar 30, 2024
a0e6ef0
doc: fix lint-md failed
congzhangzh Mar 30, 2024
05ccae1
Merge branch 'nodejs:main' into doc_buiding_speed_up_4_windows
congzhangzh Mar 30, 2024
ea09c43
Merge branch 'nodejs:main' into doc_buiding_speed_up_4_windows
congzhangzh Mar 30, 2024
28900c9
doc: minor fix
congzhangzh Mar 30, 2024
8902e84
Merge branch 'nodejs:main' into doc_buiding_speed_up_4_windows
congzhangzh Apr 4, 2024
f1725db
Merge branch 'nodejs:main' into doc_buiding_speed_up_4_windows
congzhangzh Apr 4, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
doc: fix lint-md failed
  • Loading branch information
@congzhangzh
congzhangzh committed Mar 30, 2024
commit a0e6ef0c9dc958d22aa3d23c6d1f30957162b2ad
8 changes: 4 additions & 4 deletions BUILDING.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -559,11 +559,11 @@ export LDFLAGS="-fuse-ld=mold" # add to ~/.zshrc or other shell config file

On Windows:

Tips: follow <https://github.com/ccache/ccache/wiki/MS-Visual-Studio>, and you should
notice that obj file will be bigger the normal one.
Tips: follow <https://github.com/ccache/ccache/wiki/MS-Visual-Studio>, and you
should notice that obj file will be bigger the normal one.

First, install ccache, assume ccache install to c:\ccache， copy c:\ccache\ccache.exe
to c:\ccache\cl.exe
First, install ccache, assume ccache install to c:\ccache, copy
c:\ccache\ccache.exe to c:\ccache\cl.exe

@H4ad H4ad Mar 30, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to CL.exe, is less error prone to confuse with 1.

@H4ad H4ad Mar 30, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, during the tests, I got some errors because I installed the ccache via chocolatey and I copy the shim file instead of the original file.

I think we should not include the steps to install the ccache, just ask the user to install and ensure they have the CL.exe file.

@congzhangzh congzhangzh Mar 30, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to CL.exe, is less error prone to confuse with 1.

I agree:)

@congzhangzh congzhangzh Mar 30, 2024 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, during the tests, I got some errors because I installed the ccache via chocolatey and I copy the shim file instead of the original file.

I think we should not include the steps to install the ccache, just ask the user to install and ensure they have the CL.exe file.

Yes, these instructions will make doc mess here, on another side,
As chocolatey is popular these day, I’m a little worried most of the user will do these step wrong.


```powershell
set TrackFileAccess=false
Expand Down