Bug 259965 - x11/lumina: Update to 1.6.2
Summary: x11/lumina: Update to 1.6.2
Status: Closed Works As Intended
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Loïc Bartoletti
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-11-21 15:34 UTC by Jason W. Bacon
Modified: 2022-02-23 14:15 UTC (History)
2 users (show)

See Also:
lbartoletti: maintainer-feedback+


Attachments
Unified diff updating to 1.6.1 (28.81 KB, patch)
2021-11-21 15:34 UTC, Jason W. Bacon
no flags Details | Diff
x11/lumina: update to 1.6.1 (40.93 KB, patch)
2021-12-08 22:04 UTC, Loïc Bartoletti
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jason W. Bacon freebsd_committer freebsd_triage 2021-11-21 15:34:56 UTC
Created attachment 229637 [details]
Unified diff updating to 1.6.1

Any work done yet on updating to Lumina 1.6.1?  Looks like it will not be difficult.  Attaching a patch for lumina-core.
Comment 1 Loïc Bartoletti freebsd_committer freebsd_triage 2021-11-27 07:23:06 UTC
(In reply to Jason W. Bacon from comment #0)

Thanks Jason,

I'll try to update lumina this week-end. However, lumina-core is not the only one port to update see https://www.freshports.org/search.php?stype=name&method=match&query=lumina-&num=10&orderby=category&orderbyupdown=asc&search=Search&format=html&branch=head&page=1
Comment 2 Jason W. Bacon freebsd_committer freebsd_triage 2021-11-27 13:14:04 UTC
Yes, I'm aware that there are several modules.  But -core is likely the biggest and needs to be done before the rest, so I looked into it first.

Thanks for maintaining Lumina!
Comment 3 Loïc Bartoletti freebsd_committer freebsd_triage 2021-12-08 22:04:10 UTC
Created attachment 229983 [details]
x11/lumina: update to 1.6.1

Jason,
Thanks for the patch, I made a tiny fix in post-configure. I'll look to remove the post-configure later.

BTW I updated all other ports. If you want to try them.

Loïc
Comment 4 Loïc Bartoletti freebsd_committer freebsd_triage 2021-12-08 22:04:53 UTC
My GH branch: https://github.com/lbartoletti/freebsd-ports/tree/lumina-1.6.1
Comment 5 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-11 15:24:20 UTC
Awesome, nice work...

Issues I've noticed so far:

1. CONFLICTS should probably be updated or removed
2. Changing Window Theme under Window Mnaager doesn't work
3. The default panels no longer include Start Menu, but now have Application Menu in its place.  Not technically a bug, but seems like a poor choice from my perspective since it makes Lumina look less capable.  Might be an upstream choice, but if I were maintainer I'd patch it in the port for now.
Comment 6 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-11 15:47:48 UTC
Upstream PR for window theme:
https://github.com/lumina-desktop/lumina/issues/787
Comment 7 q5sys 2021-12-19 05:48:36 UTC
(In reply to Jason W. Bacon from comment #6)
That's not a PR that's an issue.  Did you mean to file a PR?
Comment 8 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-19 15:51:58 UTC
(In reply to q5sys from comment #7)

No, I meant to reference the Github issue.
Comment 9 q5sys 2021-12-20 11:50:25 UTC
(In reply to Jason W. Bacon from comment #8)

Ok, just wanted to make sure.  I got excited for a moment that someone had PR for me that would fix something. :P
Comment 10 Loïc Bartoletti freebsd_committer freebsd_triage 2021-12-22 20:28:43 UTC
(In reply to Jason W. Bacon from comment #6)
Jason, except this issue, shall I commit the update?
Comment 11 q5sys 2021-12-22 20:37:02 UTC
@Loic
FYI, 1.6.2 will be coming out this Saturday/Sunday.  After that release there should be several months before the next one.
Comment 12 q5sys 2021-12-22 20:41:37 UTC
I'm going to try to see if I can track down that WM theme change bug that Jason reported, and if I can determine the problem (might just be a simple issue) I'll try to that in.  However If I cant figure that out quickly or I cant fix it with a quick patch that'll get pushed back to 1.6.3
Comment 13 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-22 20:45:55 UTC
(In reply to Loïc Bartoletti from comment #10)

I'd say the inability to switch window themes is a show-stopper.

There's also an issue I didn't mention here: Resizing the preferences window is incredibly slow.  The redraws lag way behind mouse movement.

When these two things are fixed, I'd say we're good to go.  I didn't notice any other serious regressions.
Comment 14 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-22 20:46:19 UTC
(In reply to q5sys from comment #12)

Cool, thanks for the heads-up.
Comment 15 q5sys 2021-12-22 20:49:40 UTC
@ Jason can you throw me a ticket on github about the preferences issue bug you've mentioned. This is the first I'm hearing of it, since I do all my ticket tracking on github.
Comment 16 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-23 01:13:21 UTC
(In reply to q5sys from comment #15)

I had meant to do that, sorry.  It's there now.
Comment 17 q5sys 2021-12-24 23:16:50 UTC
(In reply to Jason W. Bacon from comment #5)

> 2. Changing Window Theme under Window Mnaager doesn't work

This has been fixed.  

> 3. The default panels no longer include Start Menu, but now have Application Menu in its place.  Not technically a bug, but seems like a poor choice from my perspective since it makes Lumina look less capable.  Might be an upstream choice, but if I were maintainer I'd patch it in the port for now.

The application menu is the default, however the previous menu is still included.  It's one of the panel options that can be configured.  There are three types of "start menus" available in Lumina.  It can be changed using lumina-config under Interface Configuration > Panels.

"Application Menu" is the classic style of start menu that most desktops include.
"Start Menu" is what was the previous default.
"User Menu" is original start menu made for Lumina.

All three of these options are still available. In speaking with people over the years, the feedback I have gotten is that most end up switching to the Application Menu immediately upon installing the desktop.  That is why this has been set as the default, but I have not removed any of the previous menus, they are all still available. 

I have considered doing a 'first-start' option where someone chooses which they prefer, but I personally dont like being required to make decisions on a new desktop when I first start it... so I haven't done that.  However more on this later.

This menu change should only affect new installs of Lumina, as your personal lumina configuration should not be overwritten when upgrading. If you have a previous configuration you prefer, copy the ~/.config/lumina-desktop/ folder to new installs to use your preferred configuration.

To make it easier for people who use lumina and have their own unique preferences for how it should be set up.  I do have a task on my to-do list (finally made a ticket for it on github) to create an export/import utility for Lumina so it is easy for users to save/restore their custom configurations easily.
Comment 18 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-24 23:32:46 UTC
(In reply to q5sys from comment #17)

Yes, I was aware that it can be changed in the settings.

I agree that users shouldn't be burdened with another question at install time.  I don't mind manually switching it on new installations if most users prefer a simple app menu.  Just wanted to make sure this was a deliberate change and put in my vote.

Thanks...
Comment 19 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-25 00:40:52 UTC
(In reply to q5sys from comment #17)

Window theme patches were imported to my WIP port and they appear to work.

https://github.com/outpaddling/freebsd-ports-wip/tree/master/lumina-coreutils

This also resolved the slow resizing of the settings window.

I think at this point we might as well wait for 1.6.2, though.
Comment 21 Jason W. Bacon freebsd_committer freebsd_triage 2021-12-26 22:18:25 UTC
(In reply to q5sys from comment #20)

Slick.  I updated my WIP ports to 1.6.2:

https://github.com/outpaddling/freebsd-ports-wip

FYI, the screen resize issue got worse.  After resizing a vbox screen running 1.6.2, the panels disappear for good.  I'll ping the Github issue as well.
Comment 22 Loïc Bartoletti freebsd_committer freebsd_triage 2022-01-27 20:49:09 UTC
(In reply to Jason W. Bacon from comment #21)

If I'm not wrong, I know that Lumina is not suitable for VM (Virtualbox only?). Have you tried on a dedicated computer?
Comment 23 Jason W. Bacon freebsd_committer freebsd_triage 2022-01-27 23:31:58 UTC
Yeah, I have 1.6.2 running on an old Toshiba Satellite.  There don't seem to be any show-stoppers.  One remaining issue that's easily worked around be selecting a different WM theme:

https://github.com/lumina-desktop/lumina/issues/789

Correct on the vbox issue:

https://github.com/lumina-desktop/lumina/issues/536

Hopefully this one will be fixed in 1.6.3, but it's not a regression so I say go ahead with the 1.6.2 commit.
Comment 24 q5sys 2022-01-28 15:49:05 UTC
(In reply to Jason W. Bacon from comment #23)
I have been unable to reproduce the issue you're having on actual hardware.  I've spoken with a few people about it and we're trying to find a time when we can sync up to dig into what might be the issue.  But currently our schedules have not worked out.  For now, as much as I hate to say it, I'd suggest you use something other than Lumina in virtualbox.
Comment 25 commit-hook freebsd_committer freebsd_triage 2022-01-29 16:17:59 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=623f3ae564be88376786d5d67e18a017711126a4

commit 623f3ae564be88376786d5d67e18a017711126a4
Author:     Loïc Bartoletti <lbartoletti@FreeBSD.org>
AuthorDate: 2022-01-29 16:13:50 +0000
Commit:     Loïc Bartoletti <lbartoletti@FreeBSD.org>
CommitDate: 2022-01-29 16:14:50 +0000

    lumina: Update to 1.6.2

    Changes:
     - 1.6.1: https://lumina-desktop.org/post/2021-10-05/
     - 1.6.2: https://lumina-desktop.org/post/2021-12-25/

    PR:             259965
    Reported by:    jwb
    Tested by:      jwb

 deskutils/lumina-archiver/Makefile                 |   3 +-
 deskutils/lumina-archiver/distinfo                 |   6 +-
 deskutils/lumina-calculator/Makefile               |   5 +-
 deskutils/lumina-calculator/distinfo               |   6 +-
 deskutils/lumina-fileinfo/Makefile                 |   3 +-
 deskutils/lumina-fileinfo/distinfo                 |   6 +-
 deskutils/lumina-fm/Makefile                       |   3 +-
 deskutils/lumina-fm/distinfo                       |   6 +-
 deskutils/lumina-fm/pkg-plist                      |   1 -
 deskutils/lumina-mediaplayer/Makefile              |   3 +-
 deskutils/lumina-mediaplayer/distinfo              |   6 +-
 deskutils/lumina-pdf/Makefile                      |   5 +-
 deskutils/lumina-pdf/distinfo                      |   6 +-
 .../lumina-pdf/files/patch-Annotation.h (gone)     |  10 -
 .../lumina-pdf/files/patch-PrintWidget.cpp (gone)  |  11 -
 .../lumina-pdf/files/patch-drawablepage.h (gone)   |  10 -
 deskutils/lumina-photo/Makefile                    |   3 +-
 deskutils/lumina-photo/distinfo                    |   6 +-
 deskutils/lumina-screenshot/Makefile               |   3 +-
 deskutils/lumina-screenshot/distinfo               |   6 +-
 deskutils/lumina-textedit/Makefile                 |   3 +-
 deskutils/lumina-textedit/distinfo                 |   6 +-
 x11/lumina-core/Makefile                           |   9 +-
 x11/lumina-core/distinfo                           |   6 +-
 x11/lumina-core/pkg-plist                          | 472 ++++++++++++---------
 x11/lumina-coreutils/Makefile                      |   2 +-
 x11/lumina-coreutils/distinfo                      |   6 +-
 x11/lumina/Makefile                                |   2 +-
 28 files changed, 313 insertions(+), 301 deletions(-)
Comment 26 Loïc Bartoletti freebsd_committer freebsd_triage 2022-01-29 16:20:01 UTC
Committed, thanks!
Comment 27 Jason W. Bacon freebsd_committer freebsd_triage 2022-02-03 21:52:31 UTC
Seems xscreensaver is no longer needed and should probably be removed as a lumina-core dependency.
Comment 28 Jason W. Bacon freebsd_committer freebsd_triage 2022-02-09 15:41:45 UTC
(In reply to Jason W. Bacon from comment #27)

FYI: https://github.com/lumina-desktop/lumina/issues/808
Comment 29 Jason W. Bacon freebsd_committer freebsd_triage 2022-02-23 14:15:45 UTC
The new Lumina screensaver is not finished and should not have been enabled in this release.  Working on disabling it so we can continue to use xscreensaver for now.