Skip to content

build-kernel: ease copy/paste #19

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

Merged
AnneCYH merged 2 commits into from
Nov 26, 2024
Merged

build-kernel: ease copy/paste #19

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
21403f0
build-kernel: use copy/pastable commands
gagath Nov 25, 2024
4c1eafb
build-kernel: no need to +x d/rules
gagath Nov 25, 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
21 changes: 12 additions & 9 deletions docs/how-to/develop-customise/build-kernel.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -93,9 +93,9 @@ deb-src http://archive.ubuntu.com/ubuntu jammy-updates main
To install the required packages and build dependencies, run:

```{code-block} shell
sudo apt update
sudo apt build-dep -y linux linux-image-unsigned-$(uname -r)
sudo apt install -y fakeroot llvm libncurses-dev dwarves
sudo apt update && \
sudo apt build-dep -y linux linux-image-unsigned-$(uname -r) && \
sudo apt install -y fakeroot llvm libncurses-dev dwarves
```

(how-to-build-kernel-obtain-source)=
Expand Down Expand Up @@ -139,10 +139,11 @@ environment and the necessary scripts have execute permissions:

```{code-block} shell
cd <kernel_source_working_directory>
chmod a+x debian/rules
chmod a+x debian/scripts/*
chmod a+x debian/scripts/misc/*
fakeroot debian/rules clean
```
```{code-block} shell
chmod a+x debian/scripts/* && \
Copy link
Collaborator

@AnneCYH AnneCYH Nov 26, 2024

Choose a reason for hiding this comment

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

@khbecker could you take a look at this change?

@gagath , sorry, why don't we need the chmod for debian/rules?

Copy link
Contributor Author

@gagath gagath Nov 26, 2024

Choose a reason for hiding this comment

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

@AnneCYH see rationale in commit, quoting here for convenience:

The debian/rules file already has executable flag set after executing
apt source. This was verified for an old release as well (xenial).

Copy link
Collaborator

@AnneCYH AnneCYH Nov 26, 2024

Choose a reason for hiding this comment

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

ah thanks; i didn't expand the commits.
looks good then!

chmod a+x debian/scripts/misc/* && \
fakeroot debian/rules clean
```

### Modify ABI number
Expand Down Expand Up @@ -188,8 +189,10 @@ You are now ready to build the kernel.

```{code-block} shell
cd <kernel_source_working_directory>
fakeroot debian/rules clean
fakeroot debian/rules binary
```
```{code-block} shell
fakeroot debian/rules clean && \
fakeroot debian/rules binary
```

```{only} docx
Expand Down
Loading