Skip to content

Document downloading real-time sources#77

Merged
AnneCYH merged 29 commits intomainfrom
bwheeler/download-src-packages
Jan 17, 2025
Merged

Document downloading real-time sources#77
AnneCYH merged 29 commits intomainfrom
bwheeler/download-src-packages

Conversation

@benjamin051000
Copy link
Copy Markdown
Contributor

resolves #48

Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
@benjamin051000 benjamin051000 self-assigned this Jan 6, 2025
@benjamin051000 benjamin051000 changed the title Add initial how-to download real-time sources. Document downloading real-time sources Jan 6, 2025
Copy link
Copy Markdown
Member

@farshidtz farshidtz left a comment

Choose a reason for hiding this comment

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

The content looks good. Thanks.

Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 6, 2025

Everyone contributing to this PR have now signed the CLA. Thanks!

To save space and be more concise

Co-authored-by: Farshid Tavakolizadeh <farshid.tavakolizadeh@canonical.com>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
@benjamin051000 benjamin051000 force-pushed the bwheeler/download-src-packages branch from e6b673c to b802a53 Compare January 6, 2025 16:38
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Comment thread docs/how-to/download-real-time-sources.rst Outdated
I can't tell if it's possible to use headers within the `tab-item`, as
it yields a build warning and doesn't render the text. Instead, just
bold the text, which may be sufficient for now.

Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
@benjamin051000 benjamin051000 force-pushed the bwheeler/download-src-packages branch from 1901d31 to 3dbcd32 Compare January 6, 2025 19:16
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
@benjamin051000 benjamin051000 marked this pull request as ready for review January 6, 2025 19:38
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/index.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/index.rst Outdated
benjamin051000 and others added 2 commits January 10, 2025 15:10
todo squash this

Co-authored-by: AnneCYH <anne.chew@canonical.com>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
@benjamin051000 benjamin051000 force-pushed the bwheeler/download-src-packages branch from 88a5892 to ef49858 Compare January 10, 2025 20:10
installed on your system

Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 13, 2025

Everyone contributing to this PR have now signed the CLA. Thanks!

Co-authored-by: AnneCYH <anne.chew@canonical.com>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
@benjamin051000 benjamin051000 force-pushed the bwheeler/download-src-packages branch from f2e5888 to ac38f8a Compare January 13, 2025 16:01
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Copy link
Copy Markdown
Contributor

@khbecker khbecker left a comment

Choose a reason for hiding this comment

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

looks good. thanks!

Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Copy link
Copy Markdown
Collaborator

@AnneCYH AnneCYH left a comment

Choose a reason for hiding this comment

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

Hey @benjamin051000 , apologies i kept thinking this was done.
Just one final change but e'thing else looks good.

Thanks!

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 16, 2025

Everyone contributing to this PR have now signed the CLA. Thanks!

benjamin051000 and others added 2 commits January 16, 2025 08:20
Co-authored-by: AnneCYH <anne.chew@canonical.com>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Co-authored-by: AnneCYH <anne.chew@canonical.com>
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
@benjamin051000 benjamin051000 force-pushed the bwheeler/download-src-packages branch from 65a7615 to c61f216 Compare January 16, 2025 13:20
Signed-off-by: Benjamin Wheeler <benjamin.wheeler@canonical.com>
Copy link
Copy Markdown
Collaborator

@AnneCYH AnneCYH left a comment

Choose a reason for hiding this comment

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

Thanks for your effort @benjamin051000 !

Copy link
Copy Markdown
Member

@farshidtz farshidtz left a comment

Choose a reason for hiding this comment

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

Thanks for the guide.
I have a few minor suggestions which you may want to consider.

Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment thread docs/how-to/download-real-time-sources.rst Outdated
Comment on lines +72 to +76
Use the following command to add ``deb-src`` to the :file:`ubuntu-realtime-kernel.sources` file:

.. code-block:: shell

sudo sed -i '2s/^Types: deb$/Types: deb deb-src/' /etc/apt/sources.list.d/ubuntu-realtime-kernel.sources
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.

Why using sed to add the deb-src but not for uncommenting that line for 22.04?

I suggest not using sed at all and simply telling what needs be added with an example. This makes it easier to understand what is being done and to revert it.

Comment thread docs/how-to/index.rst Outdated
AnneCYH and others added 3 commits January 17, 2025 17:00
Co-authored-by: Farshid Tavakolizadeh <farshid.tavakolizadeh@canonical.com>
Signed-off-by: annecyh <anne.chew@canonical.com>
@AnneCYH AnneCYH merged commit 6782d5d into main Jan 17, 2025
@AnneCYH AnneCYH deleted the bwheeler/download-src-packages branch March 10, 2025 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Document downloading source packages

4 participants