User Details
- User Since
- Aug 19 2015, 10:52 PM (224 w, 4 d)
Yesterday
Sat, Dec 7
Tue, Dec 3
Sun, Dec 1
Sat, Nov 30
Thu, Nov 28
Tue, Nov 19
Sorry, I won’t be able to help with testing this, as I no longer have the multicast routing setup that I previously had when I reported this bug. Thanks for following up on it though!
Sun, Nov 17
Thu, Nov 14
Nov 7 2019
I am ok with this change, and provide my approval to proceed as a ports committer.
@ngie - would you like to commit this, or would you like me to? Please note the maintainer timeout discussion below.
Nov 4 2019
Nov 3 2019
Oct 26 2019
Oct 20 2019
Hi madpilot,
Oct 17 2019
Oct 16 2019
Oct 15 2019
Oct 14 2019
Oct 13 2019
Oct 9 2019
Oct 8 2019
Agree the timing is not right for this yet.
I noticed that the time-out plugin has had commits in the last 5 days to move it towards GTK3. https://git.xfce.org/panel-plugins/xfce4-time-out-plugin/log/
Oct 7 2019
No rush to this, just a thought that we should strive to:
- Minimise footprint where it doesn’t sacrifice daily use functionality - rather not install GTK twice. To that end, I was going through an exercise to see if I could have Xfce and goodies installed without GTK2 - it wasn’t easy.
- Those programs/plugins which have not moved to GTK3 yet tend to be stale/unmaintained, and should probably therefore be off by default, unless they are fundamental. Users can still elect to pkg install them manually.
Yep, agreed.
Oct 6 2019
ping - any thoughts on this propose change?
Oct 2 2019
Remove trailing \ character
Sep 22 2019
Sep 18 2019
Hi ngie,
Aug 24 2019
Aug 18 2019
This has now been committed, and this review can be closed (not sure why this wasn't automatically picked up by phabricator):
https://reviews.freebsd.org/rP509197
Aug 7 2019
One more comment on how to submit phabricator reviews - it is best if the full context of the diff is submitted (allowing me to expand the other parts of the files in between the changes if I am looking for more context).
This can be done using the "arc" command, or by generating the diff with "svn diff --diff-cmd=diff -x -U999999 > change.diff".
This is described here: https://wiki.freebsd.org/Phabricator
This only minor comment I would make, is that I prefer to keep things alphabetical in Makefile entries. I have pointed these out with inline comments, but happy for this to be committed without these being changed, as it is only a preference.
Hi Nick,
Thanks for submitting this diff - it looks like great work, and I am happy for it to be committed.
Given you are not using an @freebsd.org account here, I assume you do not have a commit bit? Would you like me to commit this on your behalf?
Regards,
Ben
Aug 4 2019
Jul 28 2019
Jun 17 2019
As per comment from madpilot, manually strip the patch from upstream,
rather than getting the ports framework to do it