diff mbox series

[2/2] ref-manual: Cleanup VOLATILE_TMP_DIR

Message ID 20241011144847.1332607-2-niko.mauno@vaisala.com
State Superseded
Headers show
Series [1/2] ref-manual: Cleanup VOLATILE_LOG_DIR | expand

Commit Message

Niko Mauno Oct. 11, 2024, 2:48 p.m. UTC
This variable was removed in
https://git.yoctoproject.org/poky/commit/?id=2f46b6f27dfa3a9d5ad177900fcecfe64c3536f1
("bitbake.conf: drop VOLATILE_TMP_DIR, use FILESYSTEM_PERMS_TABLES instead")

Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
---
 documentation/migration-guides/release-notes-4.2.rst |  2 +-
 documentation/ref-manual/variables.rst               | 12 ------------
 2 files changed, 1 insertion(+), 13 deletions(-)

Comments

Quentin Schulz Oct. 14, 2024, 10:51 a.m. UTC | #1
Hi Niko,

On 10/11/24 4:48 PM, Niko Mauno via lists.yoctoproject.org wrote:
> [You don't often get email from niko.mauno=vaisala.com@lists.yoctoproject.org. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]
> 
> This variable was removed in
> https://git.yoctoproject.org/poky/commit/?id=2f46b6f27dfa3a9d5ad177900fcecfe64c3536f1
> ("bitbake.conf: drop VOLATILE_TMP_DIR, use FILESYSTEM_PERMS_TABLES instead")
> 
> Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
> ---
>   documentation/migration-guides/release-notes-4.2.rst |  2 +-
>   documentation/ref-manual/variables.rst               | 12 ------------
>   2 files changed, 1 insertion(+), 13 deletions(-)
> 
> diff --git a/documentation/migration-guides/release-notes-4.2.rst b/documentation/migration-guides/release-notes-4.2.rst
> index 30049b89f6..ac3bb9cebe 100644
> --- a/documentation/migration-guides/release-notes-4.2.rst
> +++ b/documentation/migration-guides/release-notes-4.2.rst
> @@ -38,7 +38,7 @@ New Features / Enhancements in 4.2
> 
>   -  New variables:
> 
> -   -  :term:`VOLATILE_TMP_DIR` allows to specify
> +   -  VOLATILE_TMP_DIR allows to specify
>         whether ``/tmp`` should be on persistent storage
>         or in RAM.
> 
> diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
> index 71b3325c8f..7686e538be 100644
> --- a/documentation/ref-manual/variables.rst
> +++ b/documentation/ref-manual/variables.rst
> @@ -9983,18 +9983,6 @@ system and gives an overview of their function and contents.
> 
>            PACKAGE_INSTALL = "${INITRAMFS_SCRIPTS} ${VIRTUAL-RUNTIME_base-utils} base-passwd"
> 
> -   :term:`VOLATILE_TMP_DIR`
> -      Specifies the persistence of the target's ``/tmp`` directory.
> -
> -      By default, :term:`VOLATILE_TMP_DIR` is set to "yes", in which case
> -      ``/tmp`` links to a directory which resides in RAM in a ``tmpfs``
> -      filesystem.
> -
> -      If instead, you want the ``/tmp`` directory to be persistent, set the
> -      variable to "no" to make it a regular directory in the root filesystem.
> -
> -      This supports both sysvinit and systemd based systems.
> -

Same remark as for the other patch:
- explain how to persist /tmp (in :term:`FILESYSTEM_PERMS_TABLES` in the 
glossary for example)
- provide instructions in the migration manual so people know they need 
to stop using that variable and how to replace it while keeping the same 
behavior,

Cheers,
Quentin
Niko Mauno Oct. 15, 2024, 4:11 p.m. UTC | #2
Thanks, submitted v2 which strives to address the issues that were 
pointed out.
-Niko

On 14.10.2024 13.51, Quentin Schulz wrote:
> Hi Niko,
> 
> On 10/11/24 4:48 PM, Niko Mauno via lists.yoctoproject.org wrote:
>> [You don't often get email from 
>> niko.mauno=vaisala.com@lists.yoctoproject.org. Learn why this is 
>> important at https://aka.ms/LearnAboutSenderIdentification ]
>>
>> This variable was removed in
>> https://git.yoctoproject.org/poky/commit/?id=2f46b6f27dfa3a9d5ad177900fcecfe64c3536f1
>> ("bitbake.conf: drop VOLATILE_TMP_DIR, use FILESYSTEM_PERMS_TABLES 
>> instead")
>>
>> Signed-off-by: Niko Mauno <niko.mauno@vaisala.com>
>> ---
>>   documentation/migration-guides/release-notes-4.2.rst |  2 +-
>>   documentation/ref-manual/variables.rst               | 12 ------------
>>   2 files changed, 1 insertion(+), 13 deletions(-)
>>
>> diff --git a/documentation/migration-guides/release-notes-4.2.rst 
>> b/documentation/migration-guides/release-notes-4.2.rst
>> index 30049b89f6..ac3bb9cebe 100644
>> --- a/documentation/migration-guides/release-notes-4.2.rst
>> +++ b/documentation/migration-guides/release-notes-4.2.rst
>> @@ -38,7 +38,7 @@ New Features / Enhancements in 4.2
>>
>>   -  New variables:
>>
>> -   -  :term:`VOLATILE_TMP_DIR` allows to specify
>> +   -  VOLATILE_TMP_DIR allows to specify
>>         whether ``/tmp`` should be on persistent storage
>>         or in RAM.
>>
>> diff --git a/documentation/ref-manual/variables.rst 
>> b/documentation/ref-manual/variables.rst
>> index 71b3325c8f..7686e538be 100644
>> --- a/documentation/ref-manual/variables.rst
>> +++ b/documentation/ref-manual/variables.rst
>> @@ -9983,18 +9983,6 @@ system and gives an overview of their function 
>> and contents.
>>
>>            PACKAGE_INSTALL = "${INITRAMFS_SCRIPTS} 
>> ${VIRTUAL-RUNTIME_base-utils} base-passwd"
>>
>> -   :term:`VOLATILE_TMP_DIR`
>> -      Specifies the persistence of the target's ``/tmp`` directory.
>> -
>> -      By default, :term:`VOLATILE_TMP_DIR` is set to "yes", in which 
>> case
>> -      ``/tmp`` links to a directory which resides in RAM in a ``tmpfs``
>> -      filesystem.
>> -
>> -      If instead, you want the ``/tmp`` directory to be persistent, 
>> set the
>> -      variable to "no" to make it a regular directory in the root 
>> filesystem.
>> -
>> -      This supports both sysvinit and systemd based systems.
>> -
> 
> Same remark as for the other patch:
> - explain how to persist /tmp (in :term:`FILESYSTEM_PERMS_TABLES` in the 
> glossary for example)
> - provide instructions in the migration manual so people know they need 
> to stop using that variable and how to replace it while keeping the same 
> behavior,
> 
> Cheers,
> Quentin
diff mbox series

Patch

diff --git a/documentation/migration-guides/release-notes-4.2.rst b/documentation/migration-guides/release-notes-4.2.rst
index 30049b89f6..ac3bb9cebe 100644
--- a/documentation/migration-guides/release-notes-4.2.rst
+++ b/documentation/migration-guides/release-notes-4.2.rst
@@ -38,7 +38,7 @@  New Features / Enhancements in 4.2
 
 -  New variables:
 
-   -  :term:`VOLATILE_TMP_DIR` allows to specify
+   -  VOLATILE_TMP_DIR allows to specify
       whether ``/tmp`` should be on persistent storage
       or in RAM.
 
diff --git a/documentation/ref-manual/variables.rst b/documentation/ref-manual/variables.rst
index 71b3325c8f..7686e538be 100644
--- a/documentation/ref-manual/variables.rst
+++ b/documentation/ref-manual/variables.rst
@@ -9983,18 +9983,6 @@  system and gives an overview of their function and contents.
 
          PACKAGE_INSTALL = "${INITRAMFS_SCRIPTS} ${VIRTUAL-RUNTIME_base-utils} base-passwd"
 
-   :term:`VOLATILE_TMP_DIR`
-      Specifies the persistence of the target's ``/tmp`` directory.
-
-      By default, :term:`VOLATILE_TMP_DIR` is set to "yes", in which case
-      ``/tmp`` links to a directory which resides in RAM in a ``tmpfs``
-      filesystem.
-
-      If instead, you want the ``/tmp`` directory to be persistent, set the
-      variable to "no" to make it a regular directory in the root filesystem.
-
-      This supports both sysvinit and systemd based systems.
-
    :term:`WARN_QA`
       Specifies the quality assurance checks whose failures are reported as
       warnings by the OpenEmbedded build system. You set this variable in