dx-packages team mailing list archive

[Launchpad Bug Tracker <1999698@xxxxxxxxxxxxxxxxxx>]

This bug was fixed in the package accountsservice - 22.08.8-1ubuntu4

---------------
accountsservice (22.08.8-1ubuntu4) lunar; urgency=medium

  * debian/patches/0010-set-language.patch:
    - Fix code ambiguity causing compiler warning (LP: #1999698).

 -- Gunnar Hjalmarsson <gunnarhj@xxxxxxxxxx>  Wed, 14 Dec 2022 23:36:32
+0100

** Changed in: accountsservice (Ubuntu)
       Status: Fix Committed => Fix Released

-- 
You received this bug notification because you are a member of DX
Packages, which is subscribed to accountsservice in Ubuntu.
Matching subscriptions: dx-packages
https://bugs.launchpad.net/bugs/1999698

Title:
  Dodgy looking patch applied to our build.

Status in accountsservice package in Ubuntu:
  Fix Released

Bug description:
  When building accountsservice we apply patches, among of which is
  `0010-set-language.patch` which raises concerns.

  It will not compile without warnings: At the very least contains
  misleading indentation.

  ```
  [92/99] Compiling C object src/accounts-daemon.p/user.c.o
  ../src/user.c: In function ‘user_HOME_available’:
  ../src/user.c:1753:27: warning: this ‘else’ clause does not guard... [-Wmisleading-indentation]
   1753 |                         } else
        |                           ^~~~
  ../src/user.c:1755:33: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the ‘else’
   1755 |                                 break;
        |                                 ^~~~~
  ```

  The code:

  ```
                          } else
                                  g_free (path_to_entry);
                                  break;

  ```

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/accountsservice/+bug/1999698/+subscriptions