1. 01 Aug, 2020 2 commits
  2. 29 Jul, 2020 1 commit
  3. 13 Jul, 2020 1 commit
  4. 12 Jul, 2020 1 commit
  5. 04 Jul, 2020 1 commit
  6. 22 Jun, 2020 1 commit
  7. 13 Jun, 2020 1 commit
  8. 06 Jun, 2020 1 commit
  9. 27 May, 2020 1 commit
  10. 26 May, 2020 1 commit
  11. 23 May, 2020 4 commits
  12. 22 May, 2020 1 commit
  13. 21 May, 2020 1 commit
  14. 19 May, 2020 2 commits
  15. 09 May, 2020 1 commit
  16. 02 May, 2020 1 commit
  17. 27 Apr, 2020 3 commits
  18. 26 Apr, 2020 1 commit
    • Friedrich W. H. Kossebau's avatar
      API dox: improve docs for Q_DECLARE_FLAGS-based flags · de30ea00
      Friedrich W. H. Kossebau authored
      KApiDox & ECMAddQch have been just teached about Q_DECLARE_FLAGS, so the
      underlying typedefs are no longer skipped by doxygen, but can be
      documented now, allowing links to be generated for these types e.g. when
      used as arguments.
      
      The "#" prefix to the enum name in the description text of all the
      Q_DECLARE_FLAGS docs seems needed to properly trigger doxygen autolinks.
      
      Wording of pre-existing docs for those flags has been adapted to
      pattern used elsewhere in KF.
      
      GIT_SILENT
      de30ea00
  19. 21 Apr, 2020 1 commit
    • Stefan Brüns's avatar
      [FstabWatcher] Fix loosing of fstab watcher · 5f975738
      Stefan Brüns authored
      Summary:
      In case the fstab is deleted (caused by editing it), the watch will be
      removed. The code already partially dealed with this, i.e. it tried to
      readd the watch, but did not check if the operation succeeded and dropped
      any further changes if not.
      
      Keep a watch on the containing folder and readd the fstab on change
      events. Unfortunately QFilesystemWatcher does not allow fine granular
      change events, though as /etc/ is likely mostly silent this should not
      matter much.
      
      The bug becomes apparent when e.g. editing the fstab with VIM, which
      has a quite broken implementation of creating temporary/backup files
      during save: it moves away the original file and creates the new file
      directly under the original name, instead of using a atomic rename.
      
      Test Plan:
      1. solid-hardware listen
      2. open fstab with vi, save a few times
      Without the change, solid will loose the watch sooner or later.
      
      Depends on D28779
      
      Reviewers: #frameworks, ngraham, apol
      
      Reviewed By: ngraham, apol
      
      Subscribers: apol, kde-frameworks-devel
      
      Tags: #frameworks
      
      Differential Revision: https://phabricator.kde.org/D28780
      5f975738
  20. 13 Apr, 2020 3 commits
    • Stefan Brüns's avatar
      [Fstab] Do not emit deviceAdded twice on fstab/mtab changes · 3adf5d47
      Stefan Brüns authored
      Summary:
      The deviceAdded signal was emitted twice, once with the old device list
      in m_deviceList, once with the new one.
      
      The first one is actually wrong and was fixed in kdelibs4 by moving the
      deviceAdded signal after the m_deviceList update. In KF5, the second
      signal was added but the first one never removed.
      
      Move the m_deviceList update to the top (though for deviceRemoved it
      does not matter much), and remove the second signal.
      
      Depends on D28779
      
      Test Plan:
      1. solid-hardware listen
      2. add an entry to the fstab
      -> deviceAdded is only emitted once
      
      Reviewers: #frameworks, afiestas, apol
      
      Reviewed By: apol
      
      Subscribers: apol, kde-frameworks-devel
      
      Tags: #frameworks
      
      Differential Revision: https://phabricator.kde.org/D28782
      3adf5d47
    • Stefan Brüns's avatar
      [Fstab] Use new style connect API · 62de2826
      Stefan Brüns authored
      Summary:
      Depends on D28778
      
      GIT_SILENT
      
      Test Plan: No functional changes
      
      Reviewers: #frameworks, apol
      
      Reviewed By: apol
      
      Subscribers: kde-frameworks-devel
      
      Tags: #frameworks
      
      Differential Revision: https://phabricator.kde.org/D28779
      62de2826
    • Stefan Brüns's avatar
      [FstabWatcher] Use static QStringLiteral instead of macro · 340e1ac7
      Stefan Brüns authored
      Summary:
      QFilesystemWatcher::addPath takes a QString as argument, so avoid
      construction of tempary QString on each call.
      
      This also avoids a name clash between the Solid::Backends::Fstab::FSTAB
      logging category and the FSTAB macro.
      
      Test Plan: no functional changes
      
      Reviewers: #frameworks, apol
      
      Reviewed By: apol
      
      Subscribers: broulik, kde-frameworks-devel
      
      Tags: #frameworks
      
      Differential Revision: https://phabricator.kde.org/D28778
      340e1ac7
  21. 11 Apr, 2020 1 commit
  22. 04 Apr, 2020 1 commit
  23. 02 Apr, 2020 1 commit
    • Stefan Brüns's avatar
      [Fstab] Ensure uniqueness for all filesystem types · e468daf0
      Stefan Brüns authored
      Summary:
      Commit a99c6136 ("Samba: Ensure to differenciate mounts sharing the
      same source") only fixed the multiple mount problem for SMB share. This
      can also happen for all other mounts (remounted local file systems,
      NFS, ...).
      
      Also clarify the comment.
      
      Reviewers: #frameworks, meven, broulik, sitter
      
      Reviewed By: meven, broulik, sitter
      
      Subscribers: broulik, kde-frameworks-devel
      
      Tags: #frameworks
      
      Differential Revision: https://phabricator.kde.org/D28488
      e468daf0
  24. 01 Apr, 2020 1 commit
    • Méven Car's avatar
      Samba: Ensure to differenciate mounts sharing the same source · a99c6136
      Méven Car authored
      Summary:
      `getmntent` for samba returns fsname of the form "//server/folder" (same as `mount`)
      It is not sufficient to use in udi as device name as it does not differenciate mounts sharing the same source but having different mount points.
      
      BUG: 418906
      FIXED-IN: 5.69
      
      Test Plan:
      With a local cifs mount point:
      
      Before
      $ solid-hardware5 list | grep ourfiles
      udi = '/org/kde/fstab///meven-synapse/ourfiles'
      
      After
      $ solid-hardware5 list | grep ourfiles
      udi = '/org/kde/fstab///meven-synapse/ourfiles:/media/samba'
      
      Reviewers: #frameworks, sitter
      
      Reviewed By: sitter
      
      Subscribers: ahmadsamir, anthonyfieroni, sitter, kde-frameworks-devel
      
      Tags: #frameworks
      
      Differential Revision: https://phabricator.kde.org/D28476
      a99c6136
  25. 22 Mar, 2020 1 commit
  26. 12 Mar, 2020 1 commit
  27. 10 Mar, 2020 1 commit
    • Harald Sitter's avatar
      hardware tool: define syntax via syntax arg · 8dfb727d
      Harald Sitter authored
      Summary:
      This way the commands syntax is always immediately visible
      in the help text.
      Help that doesn't help, and
      previously the help would essentially just end up telling you to
      run --commands to actually get help -.-
      
      The existing syntax argument of addPositionArgument will simply
      print the syntax help undearneath the usage line, thus giving
      immediate access to information on how to use the application
      without first having to go through the --commands argument
      
      (--commands still exists for backwards compat)
      
      Test Plan: -h now useful
      
      Subscribers: broulik, bruns, kde-frameworks-devel
      
      Tags: #frameworks
      
      Differential Revision: https://phabricator.kde.org/D26841
      8dfb727d
  28. 07 Mar, 2020 2 commits
  29. 06 Mar, 2020 1 commit
  30. 03 Mar, 2020 1 commit