Making it prettier.
This commit is contained in:
Catalin Fratila
parent
92872154d4
commit
3b29adc000
1 changed files with 50 additions and 44 deletions
|
|
@ -13,62 +13,66 @@ on each Chromium upgrade in Electron.
|
|||
## Upgrade `libcc` to a new Chromium
|
||||
|
||||
### Steps
|
||||
1. Get the code and initialize the project:
|
||||
- ```
|
||||
$ git clone git@github.com:electron/libchromiumcontent.git
|
||||
$ cd libchromiumcontent
|
||||
$ ./script/bootstrap -v
|
||||
```
|
||||
2. Find the new beta/stable Chromium version from [OmahaProxy](https://omahaproxy.appspot.com/).
|
||||
3. Put it into the `libchromiumcontent/VERSION` file, then run `$ ./script/update`
|
||||
### 1. Get the code and initialize the project:
|
||||
```
|
||||
$ git clone git@github.com:electron/libchromiumcontent.git
|
||||
$ cd libchromiumcontent
|
||||
$ ./script/bootstrap -v
|
||||
```
|
||||
### 2. Find the new beta/stable Chromium version from [OmahaProxy](https://omahaproxy.appspot.com/).
|
||||
### 3. Put it into the `libchromiumcontent/VERSION` file, then run `$ ./script/update`
|
||||
- It will probably fail applying patches.
|
||||
4. Fix `*.patch` files in the `/patches` and `/patches-mas` folders.
|
||||
5. (Optional) Run a separate script to apply patches (`script/update` uses it internally):
|
||||
- `$ ./script/apply-patches`
|
||||
### 4. Fix `*.patch` files in the `/patches` and `/patches-mas` folders.
|
||||
### 5. (Optional) Run a separate script to apply patches (`script/update` uses it internally):
|
||||
```
|
||||
$ ./script/apply-patches
|
||||
```
|
||||
- There is also another script `/script/patch.py` that could be more useful, check `--help` to learn how it works with `$ ./script/patch.py -h`
|
||||
6. Run the build when all patches can be applied without errors
|
||||
`$ ./script/build`
|
||||
### 6. Run the build when all patches can be applied without errors
|
||||
```
|
||||
$ ./script/build
|
||||
```
|
||||
- If some patches are no longer compatible with the Chromium code, fix compilation errors.
|
||||
7. When build succeeds, create a `dist` for Electron
|
||||
### 7. When build succeeds, create a `dist` for Electron
|
||||
`$ ./script/create-dist --no_zip`
|
||||
- It will create `dist/main` folder in the root of the libcc repo, you will need it to build Electron.
|
||||
8. (Optional) Update script contents if there are errors resultant of some files being removed or renamed. (`--no_zip` prevents script from create `dist` archives, you don't need them.)
|
||||
### 8. (Optional) Update script contents if there are errors resultant of some files being removed or renamed. (`--no_zip` prevents script from create `dist` archives, you don't need them.)
|
||||
|
||||
|
||||
## Update Electron Code
|
||||
|
||||
### Steps
|
||||
1. Get the code:
|
||||
### 1. Get the code:
|
||||
```
|
||||
$ git clone git@github.com:electron/electron.git
|
||||
$ cd electron
|
||||
```
|
||||
### 2. If you already have libcc built on you machine in its own repo, you need to tell Electron explicitly to use it:
|
||||
```
|
||||
$ git clone git@github.com:electron/electron.git
|
||||
$ cd electron
|
||||
```
|
||||
2. If you already have libcc built on you machine in its own repo, you need to tell Electron explicitly to use it:
|
||||
- ```
|
||||
$ ./script/bootstrap.py -v \
|
||||
--libcc_source_path <libcc_folder>/src \
|
||||
--libcc_shared_library_path <libcc_folder>/shared_library \
|
||||
--libcc_static_library_path <libcc_folder>/static_library
|
||||
```
|
||||
- If you haven't yet built libcc but it's already supposed to be upgraded to a new Chromium, bootstrap Electron as usual
|
||||
- `$ ./script/bootstrap.py -v`
|
||||
- If you haven't yet built libcc but it's already supposed to be upgraded to a new Chromium, bootstrap Electron as usual
|
||||
`$ ./script/bootstrap.py -v`
|
||||
- Ensure that libcc submodule (`vendor/libchromiumcontent`) points to a right revision
|
||||
|
||||
3. Set CLANG_REVISION in` script/update-clang.sh` to match the version Chromium is using.
|
||||
- Located in `electron/libchromiumcontent/src/tools/clang/scripts/update.py`
|
||||
### 3. Set CLANG_REVISION in` script/update-clang.sh` to match the version Chromium is using.
|
||||
- Located in `electron/libchromiumcontent/src/tools/clang/scripts/update.py`
|
||||
|
||||
4. Checkout Chromium if you haven't already: https://chromium.googlesource.com/chromium/src.git/+/{VERSION}/tools/clang/scripts/update.py
|
||||
### 4. Checkout Chromium if you haven't already: https://chromium.googlesource.com/chromium/src.git/+/{VERSION}/tools/clang/scripts/update.py
|
||||
- (Replace the `{VERSION}` placeholder in the url above to the Chromium version libcc uses.)
|
||||
5. Build Electron.
|
||||
- Try to build Debug version first: `$ ./script/build.py -c D`
|
||||
- You will need it to run tests
|
||||
6. Fix compilation and linking errors
|
||||
7. Ensure that Release build can be built too: `$ ./script/build.py -c R`
|
||||
### 5. Build Electron.
|
||||
- Try to build Debug version first: `$ ./script/build.py -c D`
|
||||
- You will need it to run tests
|
||||
### 6. Fix compilation and linking errors
|
||||
### 7. Ensure that Release build can be built too: `$ ./script/build.py -c R`
|
||||
- Often the Release build will have different linking errors that you'll need to fix.
|
||||
- Some compilation and linking errors are caused by missing source/object files in the libcc `dist`
|
||||
8. Update `./script/create-dist` in the libcc repo, recreate a `dist`, and run Electron bootstrap script once again.
|
||||
### 8. Update `./script/create-dist` in the libcc repo, recreate a `dist`, and run Electron bootstrap script once again.
|
||||
|
||||
#### Tips for fixing compilation errors
|
||||
### Tips for fixing compilation errors
|
||||
- Fix build config errors first
|
||||
- Fix fatal errors first, like missing files and errors related to compiler flags or defines
|
||||
- Try to identify complex errors as soon as possible.
|
||||
|
|
@ -92,20 +96,22 @@ Follow all the steps above to fix Electron code on all supported platforms.
|
|||
### Steps
|
||||
If there are any compilation errors related to the Crashpad, it probably means you need to update the fork to a newer revision:
|
||||
|
||||
1. Clone Electron's fork of the Crashpad, add the main repo as another remote:
|
||||
- ```$ git clone https://github.com/electron/crashpad && cd crashpad
|
||||
### 1. Clone Electron's fork of the Crashpad, add the main repo as another remote:
|
||||
```$ git clone https://github.com/electron/crashpad && cd crashpad
|
||||
$ git remote add upstream https://chromium.googlesource.com/crashpad/crashpad/ && git fetch upstream
|
||||
```
|
||||
2. Find a revision Chromium uses in src/third_party/crashpad/README.chromium
|
||||
3. Rebase the master branch to that REVISION
|
||||
- `$ git rebase REVISION`
|
||||
### 2. Find a revision Chromium uses in src/third_party/crashpad/README.chromium
|
||||
### 3. Rebase the master branch to that REVISION
|
||||
```
|
||||
$ git rebase REVISION
|
||||
```
|
||||
- If there are nontrivial conflicts during rebase, consult with the commit's author.
|
||||
4. Add a tag with the next Electron version to a top commit in the master branch
|
||||
### 4. Add a tag with the next Electron version to a top commit in the master branch
|
||||
- e.g. if current Electron version is `1.11.*`, then use tag `electron-1.12.0`, because it will be used in the next version.
|
||||
5. Ensure Electron-specific commits in the master branch on GitHub are properly tagged
|
||||
### 5. Ensure Electron-specific commits in the master branch on GitHub are properly tagged
|
||||
- If they aren't, force push will make them orphans and they'll be lost.
|
||||
6. Force push to Electron's crashpad fork.
|
||||
7. Update `/vendor/crashpad` submodule in the Electron's upgrade branch to point to the updated crashpad.
|
||||
### 6. Force push to Electron's crashpad fork.
|
||||
### 7. Update `/vendor/crashpad` submodule in the Electron's upgrade branch to point to the updated crashpad.
|
||||
|
||||
|
||||
## Update NodeJS
|
||||
|
|
|
|||
Loading…
Reference in a new issue