[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    [patch] kdebase: kio-archive: st_uid and st_gid provides no
From:       Jarosław Staniek <js () iidea ! pl>
Date:       2007-09-26 11:03:20
Message-ID: 46FA3C78.3070908 () iidea ! pl
[Download RAW message or body]

For review:
st_uid and st_gid provides no information on win32 (as described on [1])

I think it may be possible to integrate different model for permissions on 
win32 here ant there (ACL), but now defaulting to empty UID and GID 
information seems to be enough.

[1] http://msdn2.microsoft.com/en-us/library/14h5k7ff(VS.71).aspx

-- 
regards / pozdrawiam, Jaroslaw Staniek
  Sponsored by OpenOffice Polska (http://www.openoffice.com.pl/en) to work on
  Kexi & KOffice: http://www.kexi.pl/en, http://www.koffice.org
  KDE3 & KDE4 Libraries for MS Windows: http://kdelibs.com, http://www.kde.org

["kio_archive.patch" (text/plain)]

Index: runtime/kioslave/archive/kio_archive.cpp
===================================================================
--- runtime/kioslave/archive/kio_archive.cpp	(revision 716942)
+++ runtime/kioslave/archive/kio_archive.cpp	(working copy)
@@ -129,10 +129,15 @@
         {
             archiveFile = tryPath;
             m_mtime = statbuf.st_mtime;
+#ifdef Q_WS_WIN // st_uid and st_gid provides no information
+            m_user.clear();
+            m_group.clear();
+#else
             KUser user(statbuf.st_uid);
             m_user = user.loginName();
             KUserGroup group(statbuf.st_gid);
             m_group = group.name();
+#endif
             path = fullPath.mid( pos + 1 );
             kDebug(7109).nospace() << "fullPath=" << fullPath << " path=" << path;
             len = path.length();


[prev in list] [next in list] [prev in thread] [next in thread]