firefox: add pipewire patch for wayland users #84233

colemickens · 2020-04-04T03:55:58Z

Motivation for this change

This enables WebRTC Screensharing for Firefox users on Wayland.

This patch is sourced from Fedora: https://src.fedoraproject.org/rpms/firefox/blob/master/f/firefox-pipewire-0-3.patch

an excerpt from my profiles/sway.nix:

{ pkgs, lib, config, inputs, ... }:

{
  imports = [ 
    #...
  ];
  config = {
    services.pipewire.enable = true;

    xdg.portal.enable = true;
    xdg.portal.gtkUsePortal = true;
    xdg.portal.extraPortals = with pkgs;
      [ xdg-desktop-portal-wlr xdg-desktop-portal-gtk ];

    home-manager.users.cole = { pkgs, ... }: {
      home.sessionVariables = {
        MOZ_ENABLE_WAYLAND = "1";
        MOZ_USE_XINPUT2 = "1";
        XDG_SESSION_TYPE = "wayland";
        XDG_CURRENT_DESKTOP = "sway";
      };
    home.packages = with pkgs; [ firefox ];
  };
}

(This patch is required is due to limitations at the boundary of Firefox and WebRTC build systems: (I think it's this root issue:) https://bugzilla.mozilla.org/show_bug.cgi?id=1534615)

Things done

andir · 2020-04-04T15:23:04Z

I have seen those in the past.. I have the feeling that we do not have the same bandwidth that fedora has to carry custom patches from release to release. I surely am not looking forward to rebase a patch.

What is the current situation on upstream bugzilla regarding this?

On a related note: Fedora also carried a bunch of other wayland patches last time I checked. If we go this route, would those make also sense?

colemickens · 2020-04-04T19:56:57Z

@andir

I understand wanting to minimize maintenance overhead. Some thoughts though -- this patch isn't actually from Fedora, it's from another package maintainer that maintains this patch for Nightly: https://github.com/xhorak/firefox-devedition-flatpak/blob/master/org.mozilla.FirefoxNightly/firefox-pipewire.patch. It actually just happens to apply on Stable. To me this lowers the risk - the patch is likely to be available well ahead of when it's needed here for Stable.
The issue on Bugzilla is basically that the change is wanted, but due to some complexity of their build system, is not really possible to enable upstream. I found a reddit comment that provides additional context as well. bugzilla explaining why this blocks pipewire/webrtc and the bugzilla explaining the blocking bug
I can't speak to the other Fedora patches. (maybe relevant: https://src.fedoraproject.org/rpms/firefox/tree/master)

colemickens · 2020-04-04T19:57:58Z

pkgs/applications/networking/browsers/firefox/common.nix

@@ -94,6 +94,7 @@ stdenv.mkDerivation ({
  patches = [
    ./env_var_for_system_dir.patch
  ]
+  ++ lib.optional waylandSupport ./fedora-pipewire.patch


will update the filename

Could you fetch it from fedora instead of putting it in-tree?

@jtojnar I'll update. Please note, this isn't the Fedora patch though (when I took the relevant patch from master, it didn't build, idk if our Firefox is newer?). This patch is here, which I believe was the repo building the Nightly Flatpak that most folks have been using for quite sometime: https://github.com/xhorak/firefox-devedition-flatpak

Yeah, use https://github.com/xhorak/firefox-devedition-flatpak/raw/303388734fb213a9c4e5836f5017a4a849372bdb/org.mozilla.FirefoxNightly/firefox-pipewire.patch then.

worldofpeace · 2020-04-07T23:10:01Z

I for sure want this for gnome integration 👍

worldofpeace · 2020-04-07T23:10:32Z

If this patch becomes a burden just forward the task to me.

jtojnar · 2020-04-14T00:52:39Z

There is also the issue of trademarks. We are only permitted to use the Firefox logo and name for builds that do not deviate from upstream too much, see #31843 (comment).

worldofpeace · 2020-04-14T02:41:57Z

There is also the issue of trademarks. We are only permitted to use the Firefox logo and name for builds that do not deviate from upstream too much, see #31843 (comment).

Does that mean Fedora somehow has special privileges?

I believe these are the relevant bugs in firefox:

jtojnar · 2020-04-14T02:55:47Z

No idea. Maybe legal people just did not notice yet.

Mic92 · 2020-04-22T08:12:32Z

pkgs/applications/networking/browsers/firefox/fedora-pipewire.patch

@@ -0,0 +1,473 @@
+diff --git a/config/system-headers.mozbuild b/config/system-headers.mozbuild


Not required if you use fetchpatch.

colemickens · 2020-04-22T08:16:06Z

By the way, this builds now, but I can't really validate this properly due to sway/wlroots/xdg stuff that needs a lot of attention first.

worldofpeace · 2020-04-22T12:14:07Z

@colemickens I can do this, since it should be straightforward in a GNOME environment.

worldofpeace · 2020-04-22T13:31:21Z

was getting

      |          ^~~~~~~~~~~~
free(): invalid pointer
In file included from /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/nsThreadUtils.h:18,
                 from /build/firefox-75.0/xpcom/base/nsInterfaceRequestorAgg.cpp:11,
                 from Unified_cpp_xpcom_base2.cpp:2:
/build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h: In instantiation of 'struct mozilla::detail::TupleImpl<7, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >':
/build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h:114:8:   recursively required from 'struct mozilla::detail::TupleImpl<1, bool, bool, nsILoadInfo::CrossOriginEmbedderPolicy, nsILoadInfo::CrossOriginOpenerPolicy, long unsigned int, long unsigned int, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >'
/build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h:114:8:   required from 'struct mozilla::detail::TupleImpl<0, nsTString<char16_t>, bool, bool, nsILoadInfo::CrossOriginEmbedderPolicy, nsILoadInfo::CrossOriginOpenerPolicy, long unsigned int, long unsigned int, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >'
/build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h:224:7:   required from 'class mozilla::Tuple<nsTString<char16_t>, bool, bool, nsILoadInfo::CrossOriginEmbedderPolicy, nsILoadInfo::CrossOriginOpenerPolicy, long unsigned int, long unsigned int, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >'
/build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/dom/SyncedContext.h:163:14:   required from 'struct mozilla::dom::syncedcontext::FieldStorage<void, nsTString<char16_t>, bool, bool, nsILoadInfo::CrossOriginEmbedderPolicy, nsILoadInfo::CrossOriginOpenerPolicy, long unsigned int, long unsigned int, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >'
/build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/dom/BrowsingContext.h:132:3:   required from here
/build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h:114:8: internal compiler error: Aborted
  114 | struct TupleImpl<Index, HeadT, TailT...>
      |        ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Please submit a full bug report,
with preprocessed source if appropriate.
See <https://gcc.gnu.org/bugs/> for instructions.
make[3]: *** [/build/firefox-75.0/config/rules.mk:745: Unified_cpp_xpcom_base2.o] Error 1
make[3]: *** Waiting for unfinished jobs....

andir · 2020-04-22T19:15:03Z

On 06:32 22.04.20, worldofpeace wrote: was getting ``` | ^~~~~~~~~~~~ free(): invalid pointer In file included from /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/nsThreadUtils.h:18, from /build/firefox-75.0/xpcom/base/nsInterfaceRequestorAgg.cpp:11, from Unified_cpp_xpcom_base2.cpp:2: /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h: In instantiation of 'struct mozilla::detail::TupleImpl<7, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >': /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h:114:8: recursively required from 'struct mozilla::detail::TupleImpl<1, bool, bool, nsILoadInfo::CrossOriginEmbedderPolicy, nsILoadInfo::CrossOriginOpenerPolicy, long unsigned int, long unsigned int, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >' /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h:114:8: required from 'struct mozilla::detail::TupleImpl<0, nsTString<char16_t>, bool, bool, nsILoadInfo::CrossOriginEmbedderPolicy, nsILoadInfo::CrossOriginOpenerPolicy, long unsigned int, long unsigned int, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >' /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h:224:7: required from 'class mozilla::Tuple<nsTString<char16_t>, bool, bool, nsILoadInfo::CrossOriginEmbedderPolicy, nsILoadInfo::CrossOriginOpenerPolicy, long unsigned int, long unsigned int, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >' /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/dom/SyncedContext.h:163:14: required from 'struct mozilla::dom::syncedcontext::FieldStorage<void, nsTString<char16_t>, bool, bool, nsILoadInfo::CrossOriginEmbedderPolicy, nsILoadInfo::CrossOriginOpenerPolicy, long unsigned int, long unsigned int, long unsigned int, long unsigned int, bool, bool, mozilla::dom::UserActivation::State, bool, RefPtr<mozilla::dom::FeaturePolicy>, unsigned int, nsID, bool, bool, bool, mozilla::dom::GVAutoplayRequestStatus, mozilla::dom::GVAutoplayRequestStatus, float, mozilla::dom::OrientationType, nsTString<char16_t>, mozilla::Maybe<nsTString<char16_t> > >' /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/dom/BrowsingContext.h:132:3: required from here /build/firefox-75.0/obj-x86_64-pc-linux-gnu/dist/include/mozilla/Tuple.h:114:8: internal compiler error: Aborted 114 | struct TupleImpl<Index, HeadT, TailT...> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Please submit a full bug report, with preprocessed source if appropriate. See <https://gcc.gnu.org/bugs/> for instructions. make[3]: *** [/build/firefox-75.0/config/rules.mk:745: Unified_cpp_xpcom_base2.o] Error 1 make[3]: *** Waiting for unfinished jobs.... ```

You were likely running out of memory or disk space. How much RAM and space do you have on /tmp?

worldofpeace · 2020-04-22T19:28:38Z

@andir RAM 23.1/62.8G and from the looks of df

df -h /tmp
Filesystem     Type  Size  Used Avail Use% Mounted on
zroot/root     zfs   162G  8.1G  154G   6% /

calbrecht · 2020-05-25T22:51:43Z

Fedora has a pipewire-0.3.patch now. I am currently ~~trying to build~~ building with my overlay. Also do i have the screensharing running since two months, on sway. I am interested in helping out here.

calbrecht · 2020-05-26T11:55:07Z

Also, the patch for pipewire-0.3 is the first that seems to work relatively stable, together with latest xdg-desktop-portal-wlr and latest pipewire on sway.

colemickens · 2020-05-26T19:23:41Z

@calbrecht is that from your experience in Fedora, or do you have a Firefox build from nixpkgs that works?

calbrecht · 2020-05-26T19:40:18Z

@colemickens thats from my experience from running it on nixos master since i got it to run for the first time about two months ago. See the overlay :)

berbiche · 2020-08-17T01:33:07Z

This is really awesome though only one of my monitors can be shared.
@colemickens does it work for you with multiple monitors?

alexarice · 2020-08-17T05:37:04Z

@berbiche this is a current limitation of xdp-desktop-portal-wlr

efouladi · 2020-08-26T17:42:51Z

I've updated the PR and the OP. Please see the OP for usage instructions.

This is ready for review/consideration. Thank you reviewers and those following along.

If you'd like to test (and willing to trust my binary server for this):
export NIX_PATH="nixpkgs=https://github.com/colemickens/nixpkgs/archive/nixpkgs-firefox-pipewire.tar.gz"

nix-shell \
  --option 'extra-binary-caches' 'https://cache.nixos.org https://colemickens.cachix.org' \
  --option 'trusted-public-keys' 'cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY= colemickens.cachix.org-1:bNrJ6FfMREB4bd4BOjEN85Niu8VcPdQe4F4KxVsb/I4=' \
  -p firefox --command "firefox --no-remote"
You can test on this page: https://mozilla.github.io/webrtc-landing/gum_test.html

Thanks for the great work :) I was wondering how I can use this in nixos configuration. If I import it and put it in nixpkgs.overlays I get the infinite recursion error

alexarice · 2020-08-26T17:45:15Z

I am using this overlay:

firefox-wayland = self: super: {
  firefox = let
    src = self.fetchFromGitHub {
      owner = "colemickens";
      repo = "nixpkgs";
      rev = "nixpkgs-firefox-pipewire";
      sha256 = "1yswyxy24wjxqmjld56853vq4jq98c6wmqkybnpbg537d79xplbq";
    };
    in (import src { }).firefox;
};

efouladi · 2020-08-26T17:54:40Z

I am using this overlay:

firefox-wayland = self: super: {
  firefox = let
    src = self.fetchFromGitHub {
      owner = "colemickens";
      repo = "nixpkgs";
      rev = "nixpkgs-firefox-pipewire";
      sha256 = "1yswyxy24wjxqmjld56853vq4jq98c6wmqkybnpbg537d79xplbq";
    };
    in (import src { }).firefox;
};

Thanks. this works. The sha is wrong though. On a different note, the whole reason I wanted to use this is to utilize the cache but it doesn't seem to be working. NixOS still tries to compile it from the source

alexarice · 2020-08-26T17:58:21Z

The sha was correct when I first tried this and it built from cache then, maybe it has been updated since?

efouladi · 2020-08-26T17:59:46Z

The sha was correct when I first tried this and it built from cache then, maybe it has been updated since?

yeah, that's probably it.

hlolli · 2020-09-16T13:35:38Z

I am using this overlay:

firefox-wayland = self: super: {
  firefox = let
    src = self.fetchFromGitHub {
      owner = "colemickens";
      repo = "nixpkgs";
      rev = "nixpkgs-firefox-pipewire";
      sha256 = "1yswyxy24wjxqmjld56853vq4jq98c6wmqkybnpbg537d79xplbq";
    };
    in (import src { }).firefox;
};

I tried this firefox overlay on sway, probably unrelated, but I just get a black screen and a cursor when sharing my screen. Probably configuration right? At least, there's a moving cursor, which is an improvement from the total black screen from before.

berbiche · 2020-09-16T15:03:16Z

@hlolli were you using #84233 (comment) configuration?

hlolli · 2020-09-16T15:35:56Z

@hlolli were you using #84233 (comment) configuration?

my blunder, I just added that config, now my firefox resolution is much bigger, I get a secondary screen when I enable by webcam. On google hangouts, I select to share my screen, but it's effectively a no-op, and the screen share button is greyed. But anyway, we can take this discussion somewhere else, feel it's bit off topic.

MatthewCroughan

Yes!

worldofpeace · 2020-09-21T01:43:38Z

Just rebased, I'm on GNOME 3 so I'll try it out from cachix. Will likely merge this shortly.

worldofpeace

It worked let's go.

FlorianFranzen · 2020-10-04T15:34:09Z

@NixOS/backports: Any chance we could backport this to 20.09?

worldofpeace · 2020-10-04T22:40:12Z

@NixOS/backports: Any chance we could backport this to 20.09?

I'd love that.

colemickens · 2020-10-05T01:17:04Z

Is this still working for y'all? I even pulled a newer pipewire patch from Fedora and re-built and I still can't get it to activate. (It was working when this was sent though)

berbiche · 2020-10-05T01:29:06Z

Still works on my end on revision 0b67d9e

worldofpeace · 2020-10-05T01:44:50Z

I'm using GNOME and everything at https://mozilla.github.io/webrtc-landing/gum_test.html works

worldofpeace · 2020-10-06T05:14:52Z

opened #99692

ofborg bot requested review from andir and edolstra April 4, 2020 04:03

ofborg bot added 10.rebuild-darwin: 1-10 10.rebuild-linux: 1-10 labels Apr 4, 2020

colemickens force-pushed the nixpkgs-firefox-pipewire branch from a4d16cb to 5b3bfb8 Compare April 4, 2020 05:36

colemickens changed the title ~~firefox: add fedora's pipewire patch for wayland users~~ firefox: add pipewire patch for wayland users Apr 4, 2020

colemickens commented Apr 4, 2020

View reviewed changes

flokli mentioned this pull request Apr 7, 2020

xdg-desktop-portal-wlr: init at 0.1.0 #83485

Merged

10 tasks

colemickens force-pushed the nixpkgs-firefox-pipewire branch 3 times, most recently from 029730e to f41df2b Compare April 22, 2020 07:30

Mic92 reviewed Apr 22, 2020

View reviewed changes

colemickens force-pushed the nixpkgs-firefox-pipewire branch from f41df2b to ac2d3e3 Compare April 22, 2020 08:15

colemickens force-pushed the nixpkgs-firefox-pipewire branch from ac2d3e3 to d513a7c Compare April 22, 2020 08:19

colemickens mentioned this pull request Aug 16, 2020

pipewire webrtc screencasting nix-community/nixpkgs-wayland#211

Closed

primeos mentioned this pull request Aug 20, 2020

sway: Optimal NixOS integration #57602

Open

22 tasks

colemickens force-pushed the nixpkgs-firefox-pipewire branch from 3361764 to e3c95ab Compare August 25, 2020 19:49

MatthewCroughan approved these changes Sep 21, 2020

View reviewed changes

firefox: enable pipewire+webrtc for wayland users

31e54cd

worldofpeace force-pushed the nixpkgs-firefox-pipewire branch from e3c95ab to 31e54cd Compare September 21, 2020 01:43

worldofpeace approved these changes Sep 21, 2020

View reviewed changes

worldofpeace merged commit 595d5fd into NixOS:master Sep 21, 2020

colemickens mentioned this pull request Sep 28, 2020

firefox-wayland cannot be started if it is already running #87895

Closed

worldofpeace mentioned this pull request Oct 6, 2020

[20.09] firefox: enable pipewire+webrtc for wayland users #99692

Merged

10 tasks

colemickens mentioned this pull request Oct 21, 2020

xdg-desktop-portal 1.8.0 crashing (under Sway?) #101297

Closed

ajs124 mentioned this pull request Nov 17, 2020

firefox: 82.0.3 -> 83.0 #104051

Closed

10 tasks

colemickens deleted the nixpkgs-firefox-pipewire branch December 30, 2022 01:29

		@@ -0,0 +1,473 @@
		diff --git a/config/system-headers.mozbuild b/config/system-headers.mozbuild

firefox: add pipewire patch for wayland users #84233

firefox: add pipewire patch for wayland users #84233

Conversation

colemickens commented Apr 4, 2020 • edited

Motivation for this change

Things done

andir commented Apr 4, 2020

colemickens commented Apr 4, 2020

colemickens Apr 4, 2020

Choose a reason for hiding this comment

jtojnar Apr 8, 2020

Choose a reason for hiding this comment

colemickens Apr 8, 2020

Choose a reason for hiding this comment

jtojnar Apr 9, 2020

Choose a reason for hiding this comment

worldofpeace commented Apr 7, 2020

worldofpeace commented Apr 7, 2020

jtojnar commented Apr 14, 2020

worldofpeace commented Apr 14, 2020

jtojnar commented Apr 14, 2020

Mic92 Apr 22, 2020

Choose a reason for hiding this comment

colemickens commented Apr 22, 2020

worldofpeace commented Apr 22, 2020

worldofpeace commented Apr 22, 2020

andir commented Apr 22, 2020 via email

worldofpeace commented Apr 22, 2020

calbrecht commented May 25, 2020 • edited

calbrecht commented May 26, 2020

colemickens commented May 26, 2020

calbrecht commented May 26, 2020

berbiche commented Aug 17, 2020 • edited

alexarice commented Aug 17, 2020

efouladi commented Aug 26, 2020

alexarice commented Aug 26, 2020

efouladi commented Aug 26, 2020

alexarice commented Aug 26, 2020

efouladi commented Aug 26, 2020

hlolli commented Sep 16, 2020

berbiche commented Sep 16, 2020

hlolli commented Sep 16, 2020

MatthewCroughan left a comment

Choose a reason for hiding this comment

worldofpeace commented Sep 21, 2020

worldofpeace left a comment

Choose a reason for hiding this comment

FlorianFranzen commented Oct 4, 2020

worldofpeace commented Oct 4, 2020

colemickens commented Oct 5, 2020

berbiche commented Oct 5, 2020 • edited

worldofpeace commented Oct 5, 2020

worldofpeace commented Oct 6, 2020

colemickens commented Apr 4, 2020 •

edited

calbrecht commented May 25, 2020 •

edited

berbiche commented Aug 17, 2020 •

edited

berbiche commented Oct 5, 2020 •

edited