<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>116617</bug_id>
          
          <creation_ts>2005-11-18 12:38:18 +0000</creation_ts>
          <short_desc>Please add Extended Attribute support (setxattr, getxattr)</short_desc>
          <delta_ts>2023-09-23 16:56:38 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>3</classification_id>
          <classification>Frameworks and Libraries</classification>
          <product>frameworks-kio</product>
          <component>general</component>
          <version>unspecified</version>
          <rep_platform>unspecified</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>usability</keywords>
          <priority>HI</priority>
          <bug_severity>major</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>341887</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Stephan Ferraro">saf</reporter>
          <assigned_to name="David Faure">faure</assigned_to>
          <cc>4wy78uwh</cc>
    
    <cc>alex.bikadorov</cc>
    
    <cc>aubin.ob1</cc>
    
    <cc>bugseforuns</cc>
    
    <cc>cochisecesar</cc>
    
    <cc>davidmarch007</cc>
    
    <cc>funkybomber</cc>
    
    <cc>greob</cc>
    
    <cc>grosser.meister.morti</cc>
    
    <cc>hoperidesalone</cc>
    
    <cc>ict</cc>
    
    <cc>kdelibs-bugs-null</cc>
    
    <cc>nate</cc>
    
    <cc>negora</cc>
    
    <cc>nortexoid</cc>
    
    <cc>oakad</cc>
    
    <cc>opalinsp+kde</cc>
    
    <cc>simonandric5</cc>
    
    <cc>tagwerk19</cc>
    
    <cc>tmarshall7997</cc>
    
    <cc>tobyfox</cc>
          
          <cf_commitlink>https://invent.kde.org/frameworks/kio/commit/fd0959409f356d2c7baecb849f50901123913a63</cf_commitlink>
          <cf_versionfixedin>5.76</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>391092</commentid>
    <comment_count>0</comment_count>
    <who name="Stephan Ferraro">saf</who>
    <bug_when>2005-11-18 12:38:18 +0000</bug_when>
    <thetext>Version:            (using KDE KDE 3.5.0)
Installed from:    Gentoo Packages
Compiler:          gcc 4.1 
OS:                Linux

I use a filesystem which supports extended attributes (e.g. xfs, ext3, noofs...).
Could it be possible please to add support for this attributes in konqueror?

It could be integreated for example if somebody clicks with right mouse button on a file,
that he can add meta information data on a file (meta information = extended attributes).
The extended attributes works as key:value pair.
The key is a NULL terminated string, the value can also be binary content.

Functions to use for this:
man section 2:
getxattr, lgetxattr, fgetxattr
setxattr, lsetxattr, fsetxattr

Thanks in advance.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>391307</commentid>
    <comment_count>1</comment_count>
    <who name="Thiago Macieira">thiago</who>
    <bug_when>2005-11-19 01:56:21 +0000</bug_when>
    <thetext>This has to go deeper down into kio.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>395945</commentid>
    <comment_count>2</comment_count>
    <who name="Lee Braiden">lee_b</who>
    <bug_when>2005-12-06 00:22:38 +0000</bug_when>
    <thetext>*** This bug has been confirmed by popular vote. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396044</commentid>
    <comment_count>3</comment_count>
    <who name="David Faure">faure</who>
    <bug_when>2005-12-06 09:51:04 +0000</bug_when>
    <thetext>ACL support has been implemented now, in 3.5 (mostly by Till Adam).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396162</commentid>
    <comment_count>4</comment_count>
    <who name="Lee Braiden">lee_b</who>
    <bug_when>2005-12-06 17:24:14 +0000</bug_when>
    <thetext>David, this wishlist item is for EAs, not ACLs.  I think you&apos;re just letting people know of the related feature, but I thought it best to clarify, just in case :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>396173</commentid>
    <comment_count>5</comment_count>
    <who name="David Faure">faure</who>
    <bug_when>2005-12-06 17:51:18 +0000</bug_when>
    <thetext>Ah, my bad. acl uses attr so I got confused. Reopening, sorry.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1725325</commentid>
    <comment_count>6</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2018-01-28 02:18:04 +0000</bug_when>
    <thetext>*** Bug 224635 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1725327</commentid>
    <comment_count>7</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2018-01-28 02:19:07 +0000</bug_when>
    <thetext>*** Bug 342152 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1725329</commentid>
    <comment_count>8</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2018-01-28 02:19:18 +0000</bug_when>
    <thetext>*** Bug 370371 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1725331</commentid>
    <comment_count>9</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2018-01-28 02:19:22 +0000</bug_when>
    <thetext>*** Bug 370543 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1725333</commentid>
    <comment_count>10</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2018-01-28 02:19:36 +0000</bug_when>
    <thetext>*** Bug 274327 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1725825</commentid>
    <comment_count>11</comment_count>
    <who name="">funkybomber</who>
    <bug_when>2018-01-30 08:18:30 +0000</bug_when>
    <thetext>I have just posted a bounty for this bug. I&apos;d like to see this issue fixed for both situations: 
1) when saving a file with extended file attributes
2) when copying a file/folder with extended file attributes

I know that this issue is multifaceted and serious, so this is my attempt to give it some renewed visibility and momentum!

Please note that I will need to see it solved in the KDE Neon Developer edition before I release the reward!
Happy coding! :)

relevant bug reports:
https://bugs.kde.org/show_bug.cgi?id=370543
https://bugs.kde.org/show_bug.cgi?id=274327
https://bugs.kde.org/show_bug.cgi?id=116617

bounty page:
https://www.bountysource.com/issues/54395598-please-add-extended-attribute-support-setxattr-getxattr</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1726112</commentid>
    <comment_count>12</comment_count>
    <who name="Christoph Feck">cfeck</who>
    <bug_when>2018-01-31 04:45:11 +0000</bug_when>
    <thetext>1) &quot;when saving a file with extended file attributes&quot; might be impossible to fix, because applications would need to load the attributes together with the file to be able to save them later. Are there existing applications (either by KDE or not) that keep existing extended attributes when saving? Also with &quot;Save As...&quot;?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1749384</commentid>
    <comment_count>13</comment_count>
    <who name="JKFRT">greob</who>
    <bug_when>2018-05-01 20:58:42 +0000</bug_when>
    <thetext>(In reply to Christoph Feck from comment #12)
&gt; 1) &quot;when saving a file with extended file attributes&quot; might be impossible to
&gt; fix, because applications would need to load the attributes together with
&gt; the file to be able to save them later. Are there existing applications
&gt; (either by KDE or not) that keep existing extended attributes when saving?
&gt; Also with &quot;Save As...&quot;?

Not sure if I understand the question right, but Thunar (xfce) does preserve extended attributes when copying files around.
Perhaps some of the code base can be borrowed regarding this issue?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1755900</commentid>
    <comment_count>14</comment_count>
    <who name="Christoph Feck">cfeck</who>
    <bug_when>2018-05-31 18:18:27 +0000</bug_when>
    <thetext>Having a file manager copying metadata should be possible to solve. My comment, however, was about the first situation from comment #11.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1779065</commentid>
    <comment_count>15</comment_count>
    <who name="Michael D">nortexoid</who>
    <bug_when>2018-09-05 10:57:28 +0000</bug_when>
    <thetext>(In reply to Christoph Feck from comment #12)
&gt; 1) &quot;when saving a file with extended file attributes&quot; might be impossible to
&gt; fix, because applications would need to load the attributes together with
&gt; the file to be able to save them later. Are there existing applications
&gt; (either by KDE or not) that keep existing extended attributes when saving?
&gt; Also with &quot;Save As...&quot;?

If I understand correctly, saving a file in Okular preserves xattrs. I have a bunch of files tagged (within Dolphin) that I edit in okular and save (not save as) and the xattrs remain intact. Is this a problem in other applications?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1828829</commentid>
    <comment_count>16</comment_count>
    <who name="Cochise César">cochisecesar</who>
    <bug_when>2018-12-27 17:45:17 +0000</bug_when>
    <thetext>I&apos;m working on a solution for this.
Comments are welcome.
https://phabricator.kde.org/D17816</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1899819</commentid>
    <comment_count>17</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2020-01-01 23:45:37 +0000</bug_when>
    <thetext>*** Bug 415473 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1913153</commentid>
    <comment_count>18</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2020-02-26 18:08:26 +0000</bug_when>
    <thetext>*** Bug 418225 has been marked as a duplicate of this bug. ***</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1969933</commentid>
    <comment_count>19</comment_count>
    <who name="Nate Graham">nate</who>
    <bug_when>2020-10-29 22:28:28 +0000</bug_when>
    <thetext>Git commit fd0959409f356d2c7baecb849f50901123913a63 by Nate Graham, on behalf of Gleb Popov.
Committed on 29/10/2020 at 22:28.
Pushed by ngraham into branch &apos;master&apos;.

Support for xattrs on kio copy/move

Summary:
This patch adds support to KIO to preserve xattr of files and directories on copy or move.
Working for copy/move including overwrite and rename cases.
Not working for &quot;write into&quot; when coping folders have name conflict. I&apos;m against adding to this user case as the name of the action don&apos;t imply overwrite.

Tested only on Linux.

Reviewers: dfaure, chinmoyr, bruns, #frameworks, tmarshall, usta, cochise

Reviewed By: dfaure, usta

Subscribers: usta, scheirle, tmarshall, arrowd, cfeck, bruns, phidrho, dhaumann, funkybomber, abika, pino, davidedmundson, ngraham, atha.kane, spoorun, nicolasfella, kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D17816

M  +146  -5    autotests/jobtest.cpp
M  +7    -0    autotests/jobtest.h
M  +3    -0    src/ioslaves/file/ConfigureChecks.cmake
M  +4    -1    src/ioslaves/file/config-kioslave-file.h.cmake
M  +1    -0    src/ioslaves/file/file.h
M  +126  -3    src/ioslaves/file/file_unix.cpp

https://invent.kde.org/frameworks/kio/commit/fd0959409f356d2c7baecb849f50901123913a63</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1971042</commentid>
    <comment_count>20</comment_count>
    <who name="">funkybomber</who>
    <bug_when>2020-11-02 13:08:35 +0000</bug_when>
    <thetext>I just tested this patch in KDE Neon Testing edition with:

Plasma: 5.20.2
KDE Frameworks: 5.76.0
Qt: 5.15.0
I seems to be working very well, at least when saving/updating a .txt file that already has xattrs with Kate. Well done! :)

I had less success when I tried to add some xattrs in a .zip file and then added an extra file into this .zip file using Ark. It would seem that Ark is not respecting the preservation of the xattrs during saving/updating... 

Should I file a new bug, or should this be addressed in the context of the current bug?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1971054</commentid>
    <comment_count>21</comment_count>
    <who name="Christoph Feck">cfeck</who>
    <bug_when>2020-11-02 14:22:49 +0000</bug_when>
    <thetext>It would really need to be investigated per-application. I expected this change causing a multitude of bug reports, see also comment 12.

Regarding ark, does it work if you use &quot;zip&quot; from command line? Depending on the used ark plugin, it is probably a libarchive or zip command issue. I suggest to report your findings in a new ticket.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1971238</commentid>
    <comment_count>22</comment_count>
    <who name="">funkybomber</who>
    <bug_when>2020-11-03 01:28:49 +0000</bug_when>
    <thetext>(In reply to Christoph Feck from comment #21)
&gt; It would really need to be investigated per-application. I expected this
&gt; change causing a multitude of bug reports, see also comment 12.
&gt; 
&gt; Regarding ark, does it work if you use &quot;zip&quot; from command line? Depending on
&gt; the used ark plugin, it is probably a libarchive or zip command issue. I
&gt; suggest to report your findings in a new ticket.

I tried using the zip command in the terminal and it still causes the same problem (does not preserve the xattrs).</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>