public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH manager] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist
@ 2025-09-15 12:28 Fiona Ebner
  2025-09-15 12:38 ` Fiona Ebner
  2025-11-17 17:04 ` [pve-devel] applied: " Thomas Lamprecht
  0 siblings, 2 replies; 5+ messages in thread
From: Fiona Ebner @ 2025-09-15 12:28 UTC (permalink / raw)
  To: pve-devel

Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
---
 PVE/CLI/pve_network_interface_pinning.pm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/PVE/CLI/pve_network_interface_pinning.pm b/PVE/CLI/pve_network_interface_pinning.pm
index fe803f33..a99fdafe 100644
--- a/PVE/CLI/pve_network_interface_pinning.pm
+++ b/PVE/CLI/pve_network_interface_pinning.pm
@@ -3,6 +3,7 @@ package PVE::CLI::pve_network_interface_pinning;
 use v5.36;
 
 use File::Copy;
+use File::Path qw(make_path);
 use POSIX qw(:errno_h);
 use Storable qw(dclone);
 
@@ -176,8 +177,7 @@ my sub parse_link_file {
 my $LINK_DIRECTORY = "/usr/local/lib/systemd/network/";
 
 sub ensure_link_directory_exists {
-    mkdir '/usr/local/lib/systemd' if !-d '/usr/local/lib/systemd';
-    mkdir $LINK_DIRECTORY if !-d $LINK_DIRECTORY;
+    make_path($LINK_DIRECTORY);
 }
 
 my sub get_pinned {
-- 
2.47.2



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [pve-devel] [PATCH manager] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist
  2025-09-15 12:28 [pve-devel] [PATCH manager] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist Fiona Ebner
@ 2025-09-15 12:38 ` Fiona Ebner
  2025-10-14 13:00   ` Fiona Ebner
  2025-11-17 17:04 ` [pve-devel] applied: " Thomas Lamprecht
  1 sibling, 1 reply; 5+ messages in thread
From: Fiona Ebner @ 2025-09-15 12:38 UTC (permalink / raw)
  To: pve-devel

Am 15.09.25 um 2:29 PM schrieb Fiona Ebner:
> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
> ---

Intended for both, master and stable-8.

>  PVE/CLI/pve_network_interface_pinning.pm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/PVE/CLI/pve_network_interface_pinning.pm b/PVE/CLI/pve_network_interface_pinning.pm
> index fe803f33..a99fdafe 100644
> --- a/PVE/CLI/pve_network_interface_pinning.pm
> +++ b/PVE/CLI/pve_network_interface_pinning.pm
> @@ -3,6 +3,7 @@ package PVE::CLI::pve_network_interface_pinning;
>  use v5.36;
>  
>  use File::Copy;
> +use File::Path qw(make_path);
>  use POSIX qw(:errno_h);
>  use Storable qw(dclone);
>  
> @@ -176,8 +177,7 @@ my sub parse_link_file {
>  my $LINK_DIRECTORY = "/usr/local/lib/systemd/network/";
>  
>  sub ensure_link_directory_exists {
> -    mkdir '/usr/local/lib/systemd' if !-d '/usr/local/lib/systemd';
> -    mkdir $LINK_DIRECTORY if !-d $LINK_DIRECTORY;
> +    make_path($LINK_DIRECTORY);
>  }
>  
>  my sub get_pinned {



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [pve-devel] [PATCH manager] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist
  2025-09-15 12:38 ` Fiona Ebner
@ 2025-10-14 13:00   ` Fiona Ebner
  2025-11-17 10:40     ` Fiona Ebner
  0 siblings, 1 reply; 5+ messages in thread
From: Fiona Ebner @ 2025-10-14 13:00 UTC (permalink / raw)
  To: pve-devel

Ping

Am 15.09.25 um 2:38 PM schrieb Fiona Ebner:
> Am 15.09.25 um 2:29 PM schrieb Fiona Ebner:
>> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
>> ---
> 
> Intended for both, master and stable-8.
> 
>>  PVE/CLI/pve_network_interface_pinning.pm | 4 ++--
>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/PVE/CLI/pve_network_interface_pinning.pm b/PVE/CLI/pve_network_interface_pinning.pm
>> index fe803f33..a99fdafe 100644
>> --- a/PVE/CLI/pve_network_interface_pinning.pm
>> +++ b/PVE/CLI/pve_network_interface_pinning.pm
>> @@ -3,6 +3,7 @@ package PVE::CLI::pve_network_interface_pinning;
>>  use v5.36;
>>  
>>  use File::Copy;
>> +use File::Path qw(make_path);
>>  use POSIX qw(:errno_h);
>>  use Storable qw(dclone);
>>  
>> @@ -176,8 +177,7 @@ my sub parse_link_file {
>>  my $LINK_DIRECTORY = "/usr/local/lib/systemd/network/";
>>  
>>  sub ensure_link_directory_exists {
>> -    mkdir '/usr/local/lib/systemd' if !-d '/usr/local/lib/systemd';
>> -    mkdir $LINK_DIRECTORY if !-d $LINK_DIRECTORY;
>> +    make_path($LINK_DIRECTORY);
>>  }
>>  
>>  my sub get_pinned {
> 
> 


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [pve-devel] [PATCH manager] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist
  2025-10-14 13:00   ` Fiona Ebner
@ 2025-11-17 10:40     ` Fiona Ebner
  0 siblings, 0 replies; 5+ messages in thread
From: Fiona Ebner @ 2025-11-17 10:40 UTC (permalink / raw)
  To: pve-devel

Ping

Am 14.10.25 um 3:00 PM schrieb Fiona Ebner:
> Ping
> 
> Am 15.09.25 um 2:38 PM schrieb Fiona Ebner:
>> Am 15.09.25 um 2:29 PM schrieb Fiona Ebner:
>>> Signed-off-by: Fiona Ebner <f.ebner@proxmox.com>
>>> ---
>>
>> Intended for both, master and stable-8.
>>
>>>  PVE/CLI/pve_network_interface_pinning.pm | 4 ++--
>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/PVE/CLI/pve_network_interface_pinning.pm b/PVE/CLI/pve_network_interface_pinning.pm
>>> index fe803f33..a99fdafe 100644
>>> --- a/PVE/CLI/pve_network_interface_pinning.pm
>>> +++ b/PVE/CLI/pve_network_interface_pinning.pm
>>> @@ -3,6 +3,7 @@ package PVE::CLI::pve_network_interface_pinning;
>>>  use v5.36;
>>>  
>>>  use File::Copy;
>>> +use File::Path qw(make_path);
>>>  use POSIX qw(:errno_h);
>>>  use Storable qw(dclone);
>>>  
>>> @@ -176,8 +177,7 @@ my sub parse_link_file {
>>>  my $LINK_DIRECTORY = "/usr/local/lib/systemd/network/";
>>>  
>>>  sub ensure_link_directory_exists {
>>> -    mkdir '/usr/local/lib/systemd' if !-d '/usr/local/lib/systemd';
>>> -    mkdir $LINK_DIRECTORY if !-d $LINK_DIRECTORY;
>>> +    make_path($LINK_DIRECTORY);
>>>  }
>>>  
>>>  my sub get_pinned {
>>


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [pve-devel] applied: [PATCH manager] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist
  2025-09-15 12:28 [pve-devel] [PATCH manager] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist Fiona Ebner
  2025-09-15 12:38 ` Fiona Ebner
@ 2025-11-17 17:04 ` Thomas Lamprecht
  1 sibling, 0 replies; 5+ messages in thread
From: Thomas Lamprecht @ 2025-11-17 17:04 UTC (permalink / raw)
  To: pve-devel, Fiona Ebner

On Mon, 15 Sep 2025 14:28:30 +0200, Fiona Ebner wrote:
> 


Applied, thanks!

[1/1] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist
      commit: c661af2a41cd1905a0ab2b7015816a1c7bd67fb9


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-11-17 17:08 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-09-15 12:28 [pve-devel] [PATCH manager] network-interface-pinning: fix creating directory when '/usr/local/lib' does not exist Fiona Ebner
2025-09-15 12:38 ` Fiona Ebner
2025-10-14 13:00   ` Fiona Ebner
2025-11-17 10:40     ` Fiona Ebner
2025-11-17 17:04 ` [pve-devel] applied: " Thomas Lamprecht

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal