diff mbox series

[SRU,M,N,U,1/1] UBUNTU: [Packaging] Replace fs/cifs with fs/smb/client in inclusion list

Message ID 20240405131420.182244-2-roxana.nicolescu@canonical.com
State New
Headers show
Series Include cifs.ko in linux-modules package | expand

Commit Message

Roxana Nicolescu April 5, 2024, 1:14 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/2042546

Signed-off-by: Roxana Nicolescu <roxana.nicolescu@canonical.com>
---
 debian.master/control.d/generic.inclusion-list | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Tim Gardner April 5, 2024, 1:25 p.m. UTC | #1
On 4/5/24 7:14 AM, Roxana Nicolescu wrote:
> BugLink: https://bugs.launchpad.net/bugs/2042546
> 
> Signed-off-by: Roxana Nicolescu <roxana.nicolescu@canonical.com>
> ---
>   debian.master/control.d/generic.inclusion-list | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/debian.master/control.d/generic.inclusion-list b/debian.master/control.d/generic.inclusion-list
> index af4d7381fdc9..07b533163310 100644
> --- a/debian.master/control.d/generic.inclusion-list
> +++ b/debian.master/control.d/generic.inclusion-list
> @@ -228,7 +228,7 @@ fs/binfmt_misc.ko
>   fs/btrfs/*
>   fs/cachefiles/cachefiles.ko
>   fs/ceph/*
> -fs/cifs/*
> +fs/smb/client/*

You probably want to include the whole smb directory as that also 
includes the server (which preserves the original cifs directory).

+fs/smb/*

Mantic and Noble linux-azure are already this way.

>   fs/configfs/*
>   fs/dlm/dlm.ko
>   fs/ecryptfs/*

rtg
Roxana Nicolescu April 5, 2024, 1:47 p.m. UTC | #2
On 05/04/2024 15:25, Tim Gardner wrote:
> On 4/5/24 7:14 AM, Roxana Nicolescu wrote:
>> BugLink: https://bugs.launchpad.net/bugs/2042546
>>
>> Signed-off-by: Roxana Nicolescu <roxana.nicolescu@canonical.com>
>> ---
>>   debian.master/control.d/generic.inclusion-list | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/debian.master/control.d/generic.inclusion-list 
>> b/debian.master/control.d/generic.inclusion-list
>> index af4d7381fdc9..07b533163310 100644
>> --- a/debian.master/control.d/generic.inclusion-list
>> +++ b/debian.master/control.d/generic.inclusion-list
>> @@ -228,7 +228,7 @@ fs/binfmt_misc.ko
>>   fs/btrfs/*
>>   fs/cachefiles/cachefiles.ko
>>   fs/ceph/*
>> -fs/cifs/*
>> +fs/smb/client/*
>
> You probably want to include the whole smb directory as that also 
> includes the server (which preserves the original cifs directory).
I sent a v2.
>
> +fs/smb/*
>
> Mantic and Noble linux-azure are already this way.
>
>>   fs/configfs/*
>>   fs/dlm/dlm.ko
>>   fs/ecryptfs/*
>
> rtg
diff mbox series

Patch

diff --git a/debian.master/control.d/generic.inclusion-list b/debian.master/control.d/generic.inclusion-list
index af4d7381fdc9..07b533163310 100644
--- a/debian.master/control.d/generic.inclusion-list
+++ b/debian.master/control.d/generic.inclusion-list
@@ -228,7 +228,7 @@  fs/binfmt_misc.ko
 fs/btrfs/*
 fs/cachefiles/cachefiles.ko
 fs/ceph/*
-fs/cifs/*
+fs/smb/client/*
 fs/configfs/*
 fs/dlm/dlm.ko
 fs/ecryptfs/*