[ConsoleKit] [gdm-list] Reworked patch for multi-seat and multi-display support

[Hai-Tao Halton Huo]

[resend to this correct alias] 
Hi Ray,

Thanks for your comments at
http://mail.gnome.org/archives/gdm-list/2009-May/msg00007.html

I've finished the tasks below and now the new patch is ready for review.

1 Use separate .disp file for display type, remove display-types.conf
  install .disp files to /etc/ConsoleKit/displays.d/,
  each .disp file represent a display type.
2 Only use Command in .disp file, remove Arguments/UseAuth/Chooser/Priority
3 Remove priority code
4 When adding/removing displays, CK should emit signal instead of
  calling GDM method directly
  For org.freedesktop.ConsoleKit.Seat
     add signal  SessionToAdd and SessionToRemove
     add method  ManageSeat
  For org.freedesktop.ConsoleKit.Manager
     remove method CreateStaticSessions
     add method GetUnmanagedSeats
5 Use a{sv} for dbus method CreateDisplay, since
  task #2 is done, there are only two parameters need to be send with
  signal, this work is not necessary
6 Rewrite ck-dynamic to reflect change in task #4
7 Rewrite ck_display_type_new()
8 Remove method org.freedesktop.ConsoleKit.Manager.ListCreatedSessions.
  Let ck-dynamic do the GetSeats() and filtering dynamic displays itself
9 make ck_seat_add_display(), ck_seat_set_display_type() and ck_seat_get_displ
  return a gbooolean
10 Remove unused create_display_with_parameters()
11 Comments should appear before the lines they're commenting on, not after.
12 Reverse Name and Version back
13 Figure out X11_DIR from pkg-config

The new patches links are:
* https://bugs.freedesktop.org/attachment.cgi?id=26451
  ConsoleKit part based on branch multi-seat by commit
  25145f35ab527a2219604e1710eb65e2178f6978
* http://bugzilla.gnome.org/attachment.cgi?id=135995&action=view
  GDM part based on branch display-configuration by commit
  15b783fb68e53c98d283f565eca305a3f919322d

There is only one task left - represent seat and display in
configuration files. I ever asked it before, but seems you miss that
email, please see the attached email and let me know your idea.

I saw you pushed the above changes to branches already, thanks for that.
BTW, I've got the account request for freedesktop, many thanks for pushing that.

References
-----------
[1] Design document (updated):
http://wiki.genunix.org/wiki/index.php/design_for_newgdm_consolekit_multiseat_multidisplay

[2] Test report:
http://wiki.genunix.org/wiki/index.php/testing_for_newgdm_consolekit_multiseat_multidisplay

[3] ConsoleKit bug (updated):
https://bugs.freedesktop.org/show_bug.cgi?id=19333

[4] GDM bug(updated):
http://bugzilla.gnome.org/show_bug.cgi?id=536355


Regards,
Halton.
-------------- next part --------------
An embedded message was scrubbed...
From: Halton Huo <Halton.Huo at Sun.COM>
Subject: Re: [gdm-list] Patches for multi-seat and multi-display support
Date: Fri, 15 May 2009 15:26:43 +0800
Size: 12399
Url: http://lists.freedesktop.org/archives/consolekit/attachments/20090610/c4fabedd/attachment.mht 
-------------- next part --------------
_______________________________________________
gdm-list mailing list
gdm-list at gnome.org
http://mail.gnome.org/mailman/listinfo/gdm-list