bug-gnulib@gnu.org mirror (unofficial)
 help / color / mirror / Atom feed
* [PATCH] mountlist: recognize fuse.portal as dummy file system
@ 2021-06-07 12:43 Kamil Dudka
  2021-06-07 22:16 ` Pádraig Brady
  0 siblings, 1 reply; 3+ messages in thread
From: Kamil Dudka @ 2021-06-07 12:43 UTC (permalink / raw)
  To: bug-gnulib

This was originally proposed at:

    https://lists.gnu.org/archive/html/bug-gnulib/2021-02/msg00053.html

As the full review might take some time, would it be possible to apply
at least the part related to fuse.portal file systems?  They started to
cause problems recently:

    https://bugs.launchpad.net/ubuntu/+source/xdg-desktop-portal/+bug/1905623
    https://github.com/muesli/duf/issues/35
    https://bugzilla.redhat.com/1913358
---
 lib/mountlist.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/mountlist.c b/lib/mountlist.c
index f5d1364c1b..e4c1779829 100644
--- a/lib/mountlist.c
+++ b/lib/mountlist.c
@@ -169,6 +169,7 @@
    || strcmp (Fs_type, "debugfs") == 0          \
    || strcmp (Fs_type, "devpts") == 0           \
    || strcmp (Fs_type, "fusectl") == 0          \
+   || strcmp (Fs_type, "fuse.portal") == 0      \
    || strcmp (Fs_type, "mqueue") == 0           \
    || strcmp (Fs_type, "rpc_pipefs") == 0       \
    || strcmp (Fs_type, "sysfs") == 0            \
-- 
2.31.1



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

* Re: [PATCH] mountlist: recognize fuse.portal as dummy file system
  2021-06-07 12:43 [PATCH] mountlist: recognize fuse.portal as dummy file system Kamil Dudka
@ 2021-06-07 22:16 ` Pádraig Brady
  2021-06-08  6:34   ` Kamil Dudka
  0 siblings, 1 reply; 3+ messages in thread
From: Pádraig Brady @ 2021-06-07 22:16 UTC (permalink / raw)
  To: Kamil Dudka, bug-gnulib

On 07/06/2021 13:43, Kamil Dudka wrote:
> This was originally proposed at:
> 
>      https://lists.gnu.org/archive/html/bug-gnulib/2021-02/msg00053.html
> 
> As the full review might take some time, would it be possible to apply
> at least the part related to fuse.portal file systems?  They started to
> cause problems recently:
> 
>      https://bugs.launchpad.net/ubuntu/+source/xdg-desktop-portal/+bug/1905623
>      https://github.com/muesli/duf/issues/35
>      https://bugzilla.redhat.com/1913358
> ---
>   lib/mountlist.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/lib/mountlist.c b/lib/mountlist.c
> index f5d1364c1b..e4c1779829 100644
> --- a/lib/mountlist.c
> +++ b/lib/mountlist.c
> @@ -169,6 +169,7 @@
>      || strcmp (Fs_type, "debugfs") == 0          \
>      || strcmp (Fs_type, "devpts") == 0           \
>      || strcmp (Fs_type, "fusectl") == 0          \
> +   || strcmp (Fs_type, "fuse.portal") == 0      \
>      || strcmp (Fs_type, "mqueue") == 0           \
>      || strcmp (Fs_type, "rpc_pipefs") == 0       \
>      || strcmp (Fs_type, "sysfs") == 0            \
> 

Pushed.
"fuse.portal" should be fine to always consider as dummy.

thanks,
Pádraig


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

* Re: [PATCH] mountlist: recognize fuse.portal as dummy file system
  2021-06-07 22:16 ` Pádraig Brady
@ 2021-06-08  6:34   ` Kamil Dudka
  0 siblings, 0 replies; 3+ messages in thread
From: Kamil Dudka @ 2021-06-08  6:34 UTC (permalink / raw)
  To: Pádraig Brady; +Cc: bug-gnulib

On Tuesday, June 8, 2021 12:16:36 AM CEST Pádraig Brady wrote:
> On 07/06/2021 13:43, Kamil Dudka wrote:
> > This was originally proposed at:
> >      https://lists.gnu.org/archive/html/bug-gnulib/2021-02/msg00053.html
> > 
> > As the full review might take some time, would it be possible to apply
> > at least the part related to fuse.portal file systems?  They started to
> > 
> > cause problems recently:
> >      https://bugs.launchpad.net/ubuntu/+source/xdg-desktop-portal/+bug/190
> >      5623
> >      https://github.com/muesli/duf/issues/35
> >      https://bugzilla.redhat.com/1913358
> > 
> > ---
> > 
> >   lib/mountlist.c | 1 +
> >   1 file changed, 1 insertion(+)
> > 
> > diff --git a/lib/mountlist.c b/lib/mountlist.c
> > index f5d1364c1b..e4c1779829 100644
> > --- a/lib/mountlist.c
> > +++ b/lib/mountlist.c
> > @@ -169,6 +169,7 @@
> > 
> >      || strcmp (Fs_type, "debugfs") == 0          \
> >      || strcmp (Fs_type, "devpts") == 0           \
> >      || strcmp (Fs_type, "fusectl") == 0          \
> > 
> > +   || strcmp (Fs_type, "fuse.portal") == 0      \
> > 
> >      || strcmp (Fs_type, "mqueue") == 0           \
> >      || strcmp (Fs_type, "rpc_pipefs") == 0       \
> >      || strcmp (Fs_type, "sysfs") == 0            \
> 
> Pushed.
> "fuse.portal" should be fine to always consider as dummy.

Thanks!

Kamil




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

end of thread, other threads:[~2021-06-08  6:34 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-07 12:43 [PATCH] mountlist: recognize fuse.portal as dummy file system Kamil Dudka
2021-06-07 22:16 ` Pádraig Brady
2021-06-08  6:34   ` Kamil Dudka

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).