clean up trailing whitespace in TODO file

update docs/TODO with TODOs from the source code

trivial cleanup

Remove a leftover TODO and assignment.

write HACKING file

fill out sections about subject/identity in the man page

Signed-off-by: David Zeuthen <davidz@redhat.com>

limit Change() signal storm by looking at the file monitor event

add Changed() signal that is fired when actions/authorizations change

remove debug spew for 'run' command in polkit-1

make authentication agents reconnect if polkitd-1 is restarted

add TODO item about having the authentication agent reconnect

add a TODO item about the need for file monitors

implement 'run' command for polkit-1

add TODO item about being able to cancel a CheckAuthorization() call

update TODO list

remove TODO item about moving commits to main repo as this is done

Merge branch 'master' of /home/davidz/Hacking/polkit

prepare for merge of rewritten PolicyKit

split core TODO items into separate groups for core and backend

update TODO list

start writing a TODO list

also dist org.freedesktop.ConsoleKit.xml

fix 'make distcheck'

fix up docs

finish authentication agent integration

Yay, it works!

add more infrastructure for authentication agent integration

honor implicit authorizations

Yay, only major bit remaining now is to call out to the authentication
agent on CHALLENGE...

add enumeration and handling of implicit authorizations

Now to actually use them...

implement UnregisterAuthenticationAgent()

add infrastructure for registering/unregistering authentication agents

add a hash() method to the Subject interface

remember to install polkittypes.h

first cut at authentication helper library

Not finished yet...

add pkgconfig files

add buildsys stuff for authentication library support

restrict add/remove of temporary authorizations to users

rename check_temporary_authorization_for_subject() to something more logical

make it possible to grant/revoke/use authorizations from groups

implement adding/removing temporary authorizations

implement grant and revoke for permanent authorizations

make CheckAuthorization() use permanent authorizations on disk

begin writing the implementation of the local authorization backend

add a hash() method to PolkitIdentity

forgot to add new PolkitIdentity interface

split Subject into Subject and Identity and revise API

... also remove EnumerateSessions

add remaining glue for {Add,Remove,Enumerate}Authorization

Now to finish implementing the local files backend....

rename CheckClaim() to CheckAuthorization()

Also get rid of AuthorizationClaim type and add D-Bus prototypes for a
couple of other methods.

slighty fix up docs

add checks for whether someone is authorized to check authorizations

add PolkitUnixSession and associcated functions

remove attributes from AuthorizationClaim and CheckClaim()

implement 'polkit-1 list authorizations'

add async versions of PolkitAuthority methods

rudimentary monitoring of seats and sessions

Will add public API a'la

 gboolean polkit_session_monitor_is_subject_active ();

as needed.

start writing a SessionMonitor class

add debug for each CheckClaim() call

add infrastructure for CheckClaim() method

introduce a PendingCall type to make all backend operations async

add polkit_unix_user_new_for_name()

improve output when enumerating actions using --verbose

add polkit-1 man page and rework cmd/options handling

add serialization for subjects and implement Enumerate{Users,Groups}

add subject interface and some classes implementing this interface

return NULL for GIcon if there is no icon

add --version option

bring back annotations

slightly rework commandline tool

forgot to add polkitprivate.h

forgot to add polkittypes.h

update to latest EggDBus and rework how things are done

keep up with EggDBus HEAD, in the removal of _async() and addition of sync()

http://cgit.freedesktop.org/~david/eggdbus/commit/?id=27afdc7a49ee49290e048364500d5c549f8ac614

rework AuthorizationClaim to be a GInterface so we can pass it over D-Bus

fix up namespacing for backend

the namespace is PolkitBackend so use POLKIT_BACKEND_TYPE rather than
POLKIT_TYPE_BACKEND.

port the .policy file parser and implement EnumerateActions

Also added Danish translations to test that translations work

$ dbus-send --system --print-reply --dest=org.freedesktop.PolicyKit1 /org/freedesktop/PolicyKit1/Authority org.freedesktop.PolicyKit1.Authority.EnumerateActions string:da
method return sender=:1.4481 -> dest=:1.4483 reply_serial=2
   array [
      struct {
         string "org.freedesktop.policykit.modify-defaults"
         string "Konfigurer implicit autorisering"
         string "Autentificering er påkrævet for ændre implicit autorisering"
         string "The PolicyKit Project"
         string "http://hal.freedesktop.org/docs/PolicyKit/"
         string ""
         array [
         ]
      }
      struct {
         string "org.freedesktop.policykit.read"
         string "Læs andre brugers autoriseringer"
         string "Autentificering er påkrævet for at læse andre brugers autoriseringer"
         string "The PolicyKit Project"
         string "http://hal.freedesktop.org/docs/PolicyKit/"
         string ""
         array [
            dict entry(
               string "org.freedesktop.PolicyKit.run-as-superuser.path"
               string "/usr/bin/BlahBlaster"
            )
         ]
      }
      struct {
         string "org.freedesktop.policykit.grant"
         string "Autoriser en anden bruger"
         string "Autentificering er påkrævet for at autorisere andre brugere"
         string "The PolicyKit Project"
         string "http://hal.freedesktop.org/docs/PolicyKit/"
         string ""
         array [
         ]
      }
      struct {
         string "org.freedesktop.policykit.revoke"
         string "Fjern autorisering fra en anden bruger"
         string "Autentificering er påkrævet for at fjerne en autosering fra en anden bruger"
         string "The PolicyKit Project"
         string "http://hal.freedesktop.org/docs/PolicyKit/"
         string ""
         array [
         ]
      }
   ]

$ dbus-send --system --print-reply --dest=org.freedesktop.PolicyKit1 /org/freedesktop/PolicyKit1/Authority org.freedesktop.PolicyKit1.Authority.EnumerateActions string:
method return sender=:1.4481 -> dest=:1.4484 reply_serial=2
   array [
      struct {
         string "org.freedesktop.policykit.modify-defaults"
         string "Modify defaults for implicit authorizations"
         string "Authentication is required to modify the defaults for implicit authorizations"
         string "The PolicyKit Project"
         string "http://hal.freedesktop.org/docs/PolicyKit/"
         string ""
         array [
         ]
      }
      struct {
         string "org.freedesktop.policykit.read"
         string "Read authorizations of other users"
         string "Authentication is required to read authorizations of other users"
         string "The PolicyKit Project"
         string "http://hal.freedesktop.org/docs/PolicyKit/"
         string ""
         array [
            dict entry(
               string "org.freedesktop.PolicyKit.run-as-superuser.path"
               string "/usr/bin/BlahBlaster"
            )
         ]
      }
      struct {
         string "org.freedesktop.policykit.grant"
         string "Grant authorizations to other users"
         string "Authentication is required to grant authorizations to other users"
         string "The PolicyKit Project"
         string "http://hal.freedesktop.org/docs/PolicyKit/"
         string ""
         array [
         ]
      }
      struct {
         string "org.freedesktop.policykit.revoke"
         string "Revoke authorizations from other users"
         string "Authentication is required to revoke authorizations other users"
         string "The PolicyKit Project"
         string "http://hal.freedesktop.org/docs/PolicyKit/"
         string ""
         array [
         ]
      }
   ]

add core polkit actions

add a ActionPool class and rename BackendLocal to BackendLocalAuthority

turn PolkitActionDescription into an user defined struct

Now to port the XML parser and land it in polkitbackend so backends
can easily implement EnumerateActions().

use eggdbus to generate the AuthorizationResult enumeration

fix up gtkdoc cleaning

change code style to be like GLib and generate error domain

Also port to use non-reffing structs in EggDBus HEAD.

update for arg shuffling in EggDBus HEAD

implement EnumerateUsers()

forgot to add local backend impl

start using EggDBus

fix up autogen.sh

initial commit

api cleanup (part 1)

add backend daemon

It's not right useful right now; it only says Hello() like a parrot

 $ dbus-send --print-reply --system --dest=org.freedesktop.PolicyKit.Backend / org.freedesktop.PolicyKit.Backend.Hello string:Hi!
 method return sender=:1.5180 -> dest=:1.5188 reply_serial=2
    string "You said 'Hi!'"

also remove polkitd for now

(it may return later on)

bump to version 0.90 and ensure we're parallel installable with 0.9

This is the first move towards 1.0; also

 - kill the config file
 - merge libpolkit and libpolkit-dbus

Now to write a system daemon that libpolkit will use for the
backend. Expect HEAD to be broken for a few weeks at least.

Also see http://ometer.com/parallel.html for what "parallel
installable" means. As a result, all the binaries, man pages, .policy
file dir and so forth have been renamed too. I expect the API to
change a bit. So some (not much though) porting to PolicyKit 1.0 will
be required by current users.

update NEWS again

fix 'make distcheck'

update NEWS for release

update README (for the right wiki page) and COPYING (for my email address)

fix up permissions / docs for certain helpers and files/directories

Also document in README why the various permissions/modes are
used. Thanks to Michael Biebl <biebl@debian.org> for pointing this
out.

always allow uid 0 to ask about authorizations for anyone

remove watch on fd when reaching EOF

On Wed, 2008-04-30 at 16:30 -0400, David Zeuthen wrote:
> On Wed, 2008-04-30 at 16:21 -0400, Joe Marcus Clarke wrote:
> > David Zeuthen wrote:
> > > On Wed, 2008-04-30 at 13:52 -0400, Joe Marcus Clarke wrote:
> > >> David Zeuthen wrote:
> > >>> On Wed, 2008-04-30 at 13:44 -0400, Joe Marcus Clarke wrote:
> > >>>> Correct.  I think it does read all the data, then the stream puts out
> > >>>> EOF which causes the helper to be called ad infinitum.
> > >>> I see. Mmm.. In that case, polkit_grant_io_func() can detect EOF
> > >>> (getline() returning -1) and then call the remove_watch() method
> > >>> supplied by polkit-gnome-manager.c right?
> > >> It could, but what's the difference between that and returning FALSE in
> > >> the caller?  Both would result in the watch being removed.  And since
> > >> the io_func reads the entire amount of data (until EOF) that shouldn't
> > >> be a problem.
> > >
> > > The PAM conversation happens over that fd and for some cases I don't
> > > think we read all the data at once. So there may be multiple calls to
> > > polkit_grant_io_func(). As such, returning FALSE won't work for all
> > > cases.
> >
> > It looks like it will (read all data).  The polkit_grant_io_func() runs
> > in a while loop waiting for readline to return -1.  I don't see that
> > this function ever returns to the caller until readline returns -1
> > (signifying EOF or some other error).  Am I misunderstanding something?
>
> No, you're right, that's how it works right now. But in the future this
> function might return control back to the application and then it needs
> to be called again by the watch when the application goes to process the
> main loop.
>
> Anyway, the other main reason I'd like polkit_grant_io_func() to call
> remove_watch() as opposed to making io_watch_have_data() in
> polkit-gnome-manager.c return FALSE is because of the fact that
> PolicyKit-gnome is just one of many users of libpolkit-grant (others
> right now are: polkit-auth(1), the PolicyKit-kde project that some
> people are working on)

That works.  Adding this hunk to polkit-grant.c fixes the problem:

@@ -419,6 +420,8 @@ polkit_grant_io_func (PolKitGrant *polki

         if (line != NULL)
                 free (line);
+
+        polkit_grant->func_remove_watch (polkit_grant, polkit_grant->io_watch_id);
 }

 /**

Joe

fix typo

fix autotools screwup

I hate autotools.

fix some build stuff to so the test suites run

need to link with libkit.la for some helpers