Inconsistent use of brackets in /usr/lib/apt/apt.systemd.daily

Bug #1581985 reported by Removed by request
6
Affects Status Importance Assigned to Milestone
apt (Ubuntu)
Fix Released
Low
Julian Andres Klode

Bug Description

I'm using Ubuntu 16.10 dev with apt 1.3~exp1 and on making a look at /usr/lib/apt/apt.systemd.daily I have noticed that in line 293 the opening bracket for the function check_power is used within the same line while other functions use it in the next line.

Revision history for this message
Julian Andres Klode (juliank) wrote :

For the reference, that is:

""""
check_power(){
"""
vs

"""
debug_echo()
{
"""

elsewhere.

Changed in apt (Ubuntu):
assignee: nobody → Julian Andres Klode (juliank)
importance: Undecided → Low
status: New → Triaged
Revision history for this message
Julian Andres Klode (juliank) wrote :

I have committed the fix in the APT repository, it will be part of the next APT release, I think probably early next week.

URL: https://anonscm.debian.org/cgit/apt/apt.git/commit/?id=ea4c6b3

Commit message:

commit ea4c6b3ec839976c205c05984c2a64b77bac5317
Author: Julian Andres Klode <email address hidden>
Date: Sun May 15 18:11:48 2016 +0200

    apt.systemd.daily: Put opening brace of check_power on extra line

    The rest is also on the same line, so let's go consistent here
    now that we have a bug report about it.

    LP: #1581985

Changed in apt (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package apt - 1.3~pre2ubuntu4

---------------
apt (1.3~pre2ubuntu4) yakkety; urgency=medium

  * Cherry-pick the following commits from rebased branch:
    - gpgv: Unlink the correct temp file in error case
    - Allow to specify - as APT_CONFIG for stdin
    - gpgv: Pass current config to apt-key on a file-based stdin
   [This partially reverts ubuntu3]
   (LP: #1607283)

 -- Julian Andres Klode <email address hidden> Wed, 03 Aug 2016 14:52:04 +0000

Changed in apt (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.