extra "fi" in file

Bug #1266496 reported by erik
This bug report is a duplicate of:  Bug #1315442: Extra "fi" in /etc/init.d/gdm. Edit Remove
30
This bug affects 6 people
Affects Status Importance Assigned to Milestone
gdm (Ubuntu)
Triaged
Low
Tim Lunn

Bug Description

In file /etc/init.d/gdm, line 79, there is an extra "fi" without any matching "if" inside function gen_config():

gen_config() {
  # GSettings configuration uses dconf update to generate a gdm profile
  needed=no
  if [ ! -f /etc/dconf/db/gdm ]; then
      needed=yes
  else
      for f in \
          /etc/dconf/db/gdm.d /etc/dconf/db/gdm.d/*-settings \
          /etc/dconf/db/gdm.d/locks /etc/dconf/db/gdm.d/locks/*-locks \
          /etc/gdm/greeter.gsettings; do
          if [ "$f" -nt /etc/dconf/db/gdm ]; then
              needed=yes
              break
          fi
      done
  fi
  if [ "$needed" = yes ]; then
      # The configuration file in /etc uses org.gnome.blah gsettings syntax.
      # Convert it to org/gnome/blah dconf syntax, if needed (newer)
      dist_settings=/etc/dconf/db/gdm.d/90-debian-settings
      if [ ! -f $dist_settings ] || \
          [ /etc/gdm/greeter.gsettings -nt $dist_settings ] ; then
          awk '/\[.*\]/ { gsub("\\.","/"); } ! /^#/ { print;}' \
              /etc/gdm/greeter.gsettings > $dist_settings
      fi
      echo "Updating dconf for gdm..."
      dconf update
    fi
  fi
}

Second-last "fi" doesn't match any "if".

This error is present in gdm source package 3.8.4-0ubuntu2

My version of Ubuntu:
Description: Ubuntu 13.10
Release: 13.10

My version of gdm:
gdm:
  Installed: 3.8.4-0ubuntu2
  Candidate: 3.8.4-0ubuntu2

Tim Lunn (darkxst)
Changed in gdm (Ubuntu):
importance: Undecided → Low
assignee: nobody → Tim (darkxst)
milestone: none → ubuntu-14.04
status: New → Triaged
Revision history for this message
littlebat (dashing-meng) wrote :

This bug still exists in ubuntu GNOME 14.04.

Mathew Hodson (mhodson)
Changed in gdm (Ubuntu):
milestone: ubuntu-14.04 → none
tags: added: bitesize saucy trusty
Revision history for this message
Lucy Llewellyn (lucyllewy) wrote :

This issue is for Trusty Tahr, which ended mainstream support in 2019 and exits ESM in 2022. I vote we close this issue as WONTFIX for those distributions (I don't have permission to set WONTFIX status on this issue).

Is this still relevant on any currently-supported releases of Ubuntu?

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.