New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update qubes-desktop-linux-i3 package to version 4.15 #3781

Closed
anadahz opened this Issue Apr 3, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@anadahz

anadahz commented Apr 3, 2018

Qubes OS version: R4.0

Affected component(s): qubes-desktop-linux-i3

The current i3 package version in Qubes R4.0 is too old (version 4.12), I prepared a patch and bumped up to the newer and recommended i3 version 4.15.

I'm running into a compilation error so I'm not sure if I miss some dependencies or anything, below the error excerpt of make rpms:

  CC       src/i3-handlers.o
../../i3-4.15/src/handlers.c: In function 'handle_client_message':
../../i3-4.15/src/handlers.c:818:31: error: 'A__NET_REQUEST_FRAME_EXTENTS' undeclared (first use in this function)
     } else if (event->type == A__NET_REQUEST_FRAME_EXTENTS) {
                               ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../i3-4.15/src/handlers.c:818:31: note: each undeclared identifier is reported only once for each function it appears in
../../i3-4.15/src/handlers.c:844:13: error: 'A__NET_FRAME_EXTENTS' undeclared (first use in this function); did you mean 'A__NET_REQUEST_FRAME_EXTENTS'?
             A__NET_FRAME_EXTENTS,
             ^~~~~~~~~~~~~~~~~~~~
             A__NET_REQUEST_FRAME_EXTENTS
../../i3-4.15/src/handlers.c: In function 'handle_motif_hints_change':
../../i3-4.15/src/handlers.c:1309:87: error: 'A__MOTIF_WM_HINTS' undeclared (first use in this function); did you mean 'XCB_ATOM_WM_HINTS'?
                                                                        false, window, A__MOTIF_WM_HINTS, XCB_GET_PROPERTY_TYPE_ANY, 0, 5 * sizeof(uint64_t)),
                                                                                       ^~~~~~~~~~~~~~~~~
                                                                                       XCB_ATOM_WM_HINTS
../../i3-4.15/src/handlers.c: In function 'property_handlers_init':
../../i3-4.15/src/handlers.c:1452:34: error: 'A__MOTIF_WM_HINTS' undeclared (first use in this function); did you mean 'XCB_ATOM_WM_HINTS'?
     property_handlers[10].atom = A__MOTIF_WM_HINTS;
                                  ^~~~~~~~~~~~~~~~~
                                  XCB_ATOM_WM_HINTS
make[3]: *** [Makefile:2281: src/i3-handlers.o] Error 1

The changes can be found in the following repository:
https://github.com/anadahz/qubes-desktop-linux-i3/tree/update/i3-4.15

Happy to provide any further details and bring the update to the latest i3 version forward.

@marmarek

This comment has been minimized.

Show comment
Hide comment
@marmarek

marmarek Apr 3, 2018

Member

Building i3-4.15 package from Fedora 26, on Fedora 25 (which should match what we have in dom0) works:

mock  -r fedora-25-x86_64 --rebuild i3-4.15-1.fc26.src.rpm

You may want to look at that package. On first sight looks pretty similar, but there may be some details.

Also note that just make rpms do not update dependencies. rpmbuild should fail if some are missing, but will not install/update them for you. You may want to either build src.rpm (make srpms) and use mock (see above), or use qubes-builder.

Member

marmarek commented Apr 3, 2018

Building i3-4.15 package from Fedora 26, on Fedora 25 (which should match what we have in dom0) works:

mock  -r fedora-25-x86_64 --rebuild i3-4.15-1.fc26.src.rpm

You may want to look at that package. On first sight looks pretty similar, but there may be some details.

Also note that just make rpms do not update dependencies. rpmbuild should fail if some are missing, but will not install/update them for you. You may want to either build src.rpm (make srpms) and use mock (see above), or use qubes-builder.

@anadahz

This comment has been minimized.

Show comment
Hide comment
@anadahz

anadahz Apr 4, 2018

The issue was on the rebase of the qubes patch on the newer i3 version.
Fixed and opened PR: QubesOS/qubes-desktop-linux-i3#13

Feel free to close this issue.

anadahz commented Apr 4, 2018

The issue was on the rebase of the qubes patch on the newer i3 version.
Fixed and opened PR: QubesOS/qubes-desktop-linux-i3#13

Feel free to close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment