Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 94939 - Use an xmlified devaway on www.g.o
Summary: Use an xmlified devaway on www.g.o
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Infrastructure
Classification: Unclassified
Component: Web Node Issues (show other bugs)
Hardware: All Other
: High normal
Assignee: Xavier Neys (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-06-03 11:33 UTC by Xavier Neys (RETIRED)
Modified: 2005-07-26 05:20 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Xavier Neys (RETIRED) gentoo-dev 2005-06-03 11:33:24 UTC
I tweaked the current index.php to export devaway as xml.
It's at http://dev.gentoo.org/~neysx/devaway/xml/

If you find it useful, maybe you could move it to /devaway/xml/
Don't forget to change $file = "/home/neysx/dev.Status"; in my index.php to use the official copy ("/var/dev_away/dev.Status")

Besides, if you could make the web nodes download the xml file to /dyn, we could easily use it. xml/xsl are ready, see POC at
http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml
http://www.gentoo.org/proj/en/gdp/tests/devaway.xml
Comment 1 Kurt Lieber (RETIRED) gentoo-dev 2005-07-07 08:43:23 UTC
ok, partially done:

http://dev.gentoo.org/devaway/xml/index.php

still working on getting it out to the web nodes.
Comment 2 Kurt Lieber (RETIRED) gentoo-dev 2005-07-07 08:56:35 UTC
per msg sent to -infra:
--------------------------------------
I changed cf.gweb and wrote a small script to download the devaway file
from toucan to each of the www nodes.

If something blows up with the nightly run, just comment out line 310 in
cf.gweb and all changes will be backed out.

This is per bug 94939

--kurt
--------------------------------------

will wait a couple days to make sure it's working as expected before closing.
Comment 3 Kurt Lieber (RETIRED) gentoo-dev 2005-07-08 09:13:00 UTC
it's not working -- xavier, can you take a peek?  it doesn't appear to be
picking up the XSL/DTD files.

http://www.gentoo.org/dyn/devaway/devaway.xml

Comment 4 Xavier Neys (RETIRED) gentoo-dev 2005-07-08 09:57:04 UTC
It works nicely.
/dyn/devaway/devaway.xml is not meant to be transformed(1) but both
http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml
http://www.gentoo.org/proj/en/gdp/tests/devaway.xml use it.
I'll bug devrel in a few days to ask them to integrate it into the roll-call.


(1) /dyn/devaway/devaway.xml or /dyn/news-index.xml and others under dyn are
actually processed by guide.xsl at it is the default xsl all xml files go
through, and the result is the concatenation of all text nodes.
Comment 5 Xavier Neys (RETIRED) gentoo-dev 2005-07-12 01:46:41 UTC
devrel, 

could you please have a look at
 http://www.gentoo.org/proj/en/gdp/tests/devaway.xml
 http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml
and answer the following questions:

1) is it OK if I put devaway.x{m,s}l next to the userlist under
/proj/en/devrel/roll-call ?
2) is it OK if I link it from the curent userlist.xml ?
3) is it OK if userlist.xml shows away devs as shown in my test page ?

4) bit off-topic: would it be OK if I edited some GDP member's info (eg: missing
email for bennyc)?

Thanks.
Comment 6 Donnie Berkholz (RETIRED) gentoo-dev 2005-07-12 02:26:59 UTC
I don't like how they're shown as away. It's very small and likely to be missed.
Can you do something that will stand out more, be more obvious?
Comment 7 Xavier Neys (RETIRED) gentoo-dev 2005-07-12 04:51:31 UTC
(In reply to comment #6)
> I don't like how they're shown as away. It's very small and likely to be missed.
> Can you do something that will stand out more, be more obvious?

Sure, I just committed some examples at
http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml

IMO, <brite> is not readable and using <ti> instead of <th> does not look nice.
Just appending (1) or (a) or (away) seems better.
Any other idea?
Comment 8 Donnie Berkholz (RETIRED) gentoo-dev 2005-07-12 11:37:49 UTC
(In reply to comment #7)
> Sure, I just committed some examples at
> http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml
> 
> IMO, <brite> is not readable and using <ti> instead of <th> does not look nice.
> Just appending (1) or (a) or (away) seems better.
> Any other idea?

Even (away) is better than that tiny (1). So far, (away) and <ti> are the best.

Other options are adding a new column for status, or pulling away people from
that page altogether and having a separate page for them, as we do for retired
people.
Comment 9 Xavier Neys (RETIRED) gentoo-dev 2005-07-16 02:52:09 UTC
How about using an (away) link in the name column instead?
The link points to the devaway page where the dev's status is highlighted.
http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml
Comment 10 Donnie Berkholz (RETIRED) gentoo-dev 2005-07-16 13:11:27 UTC
The more I think about it, the more it looks like all these solutions are just
hacks for the real need for an extra column with status in it.
Comment 11 Xavier Neys (RETIRED) gentoo-dev 2005-07-17 00:21:39 UTC
(In reply to comment #10)
> The more I think about it, the more it looks like all these solutions are just
> hacks for the real need for an extra column with status in it.

Agreed. I was a bit reluctant to add an extra column in an already too large
table. If it's ok with you, http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml

Comment 12 Donnie Berkholz (RETIRED) gentoo-dev 2005-07-18 00:08:34 UTC
(In reply to comment #11)
> Agreed. I was a bit reluctant to add an extra column in an already too large
> table. If it's ok with you, http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml

Heh, sometimes the best solution still isn't a good one. Could that column
either immediately follow the name or be at the far right?
Comment 13 Xavier Neys (RETIRED) gentoo-dev 2005-07-19 02:33:08 UTC
(In reply to comment #12)
> Heh, sometimes the best solution still isn't a good one. Could that column
> either immediately follow the name or be at the far right?

Sure. I put the status column right after the name like it was already on the
retired devs page.
http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml
Comment 14 Tim Yamin (RETIRED) gentoo-dev 2005-07-22 08:20:26 UTC
I also don't like the extra column. I would simply add the away information in
brackets to the "Areas of responsibility" column (and possibly make it red or
something).

Regarding your questions:

1. When it's fixed up, sure.
2. When it's fixed up, sure also.
3. See above.
4. If you can do it, yes, otherwise send me a patch.
Comment 15 Tim Yamin (RETIRED) gentoo-dev 2005-07-22 08:23:46 UTC
(In reply to comment #14)
> I also don't like the extra column. I would simply add the away information in
> brackets to the "Areas of responsibility" column (and possibly make it red or
> something).

Having said that, it is a good idea for ?statusFilter=X where we'd actually be
filling the column up with Gone/Retired/Whatever. It just happens that with
active devs we're wasting space and mostly blank columns don't look good visually.

Thanks!
Comment 16 Tim Yamin (RETIRED) gentoo-dev 2005-07-22 08:25:59 UTC
Also, more of an infra thing: it would be funky if we could have this (once
fixed up) aliased to www.gentoo.org/devaway or something as well as a redirect
from the current dev.gentoo.org/devaway page to the new one.
Comment 17 Donnie Berkholz (RETIRED) gentoo-dev 2005-07-22 11:06:13 UTC
(In reply to comment #14)
> I also don't like the extra column. I would simply add the away information in
> brackets to the "Areas of responsibility" column (and possibly make it red or
> something).

This makes it much harder to actually scan the list for people who are away.
Although whether that's a common task is harder to determine, except for the
people who periodically go through and look for away's that should be gone. What
we really need is a way to sort by status for away (what we've got for retired
now), instead of getting the whole list.
Comment 18 Donnie Berkholz (RETIRED) gentoo-dev 2005-07-22 11:06:41 UTC
(In reply to comment #16)
> Also, more of an infra thing: it would be funky if we could have this (once
> fixed up) aliased to www.gentoo.org/devaway or something as well as a redirect
> from the current dev.gentoo.org/devaway page to the new one.

Then we'll need to print the actual away messages somewhere too.
Comment 19 Xavier Neys (RETIRED) gentoo-dev 2005-07-22 11:41:14 UTC
The whole point of a devaway page is to list the devs who are away. It exists
and would be linked from userlist.xml.
Having an indication that a dev is away on the userlist itself should be an
indication, nothing more. I thought it would be useful, maybe I should have stfu.
I like the idea of adding [away] in the last column instead of wasting one on that.
You can have it in <brite> (ie. bold red) or as a link to the devaway page where
the selected devaway message is shown in <brite>.
You can also decide not to put any away indication.

Please let me know your final vote on this.

Still with an extra col, it's trivial to display [away] in last column instead:
http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml
Devaway page: http://www.gentoo.org/proj/en/gdp/tests/devaway.xml
Devaway page with highlighted dev:
http://www.gentoo.org/proj/en/gdp/tests/devaway.xml?select=absinthe
FYI, devaway data source http://dev.gentoo.org/devaway/xml
Comment 20 Donnie Berkholz (RETIRED) gentoo-dev 2005-07-22 11:49:16 UTC
I can deal with tim's idea of putting a highlighted away in the activities or
whatever.

Is there some chance we could get that link to the devaway page anchored, so the
right person ends up on the screen? If you click on zypher on the main dev list,
he's not even on the screen when the away list opens up.
Comment 21 Xavier Neys (RETIRED) gentoo-dev 2005-07-22 11:57:53 UTC
(In reply to comment #20)
> I can deal with tim's idea of putting a highlighted away in the activities or
> whatever.

Good the read that.

> Is there some chance we could get that link to the devaway page anchored, so the
> right person ends up on the screen? If you click on zypher on the main dev list,
> he's not even on the screen when the away list opens up.

I had already thought of that :)
I'd need to change GuideXML to allow ID'ing table rows with <tr id="here"> like
we already do for chapters & sections.
It's trivial to do but I don't want to change GuideXML (ie. dtd + xsl + GuideXML
doc) unless I'm sure it's going to be used.
Comment 22 Xavier Neys (RETIRED) gentoo-dev 2005-07-24 14:44:17 UTC
If we can agree on http://www.gentoo.org/proj/en/gdp/tests/userinfo.xml , I'll
make anchoring work on table rows.
"away" links from the user list to the devaway page look like
http://www.gentoo.org/proj/en/gdp/tests/devaway.xml?select=absinthe#absinthe

Just pretend the #nickname anchor is already working :)
Comment 23 Tim Yamin (RETIRED) gentoo-dev 2005-07-25 17:51:53 UTC
Nice work, looks great :)! If you want to get that placed into roll-call as it
now that's fine with me -- feel free to do it unless you want me to.
Comment 24 Jan Kundrát (RETIRED) gentoo-dev 2005-07-26 03:11:10 UTC
It would be good if the anchor pointing to the developer's away status included 
the Away Message as title attribute. 
 
And if the rows in table were grey for absent developers, it would improve 
readability (on graphical browsers :-) ). 
Comment 25 Xavier Neys (RETIRED) gentoo-dev 2005-07-26 03:45:08 UTC
(In reply to comment #23)
> Nice work, looks great :)! If you want to get that placed into roll-call as it
> now that's fine with me -- feel free to do it unless you want me to.

I'm doing it. I also need to allow <tr id="toto> in guidexml.
Comment 26 Xavier Neys (RETIRED) gentoo-dev 2005-07-26 05:20:44 UTC
(In reply to comment #16)
> Also, more of an infra thing: it would be funky if we could have this (once
> fixed up) aliased to www.gentoo.org/devaway or something as well as a redirect
> from the current dev.gentoo.org/devaway page to the new one.

Note to infra: if you rewrite dev.gentoo.org/devaway, please remember that
dev.gentoo.org/devaway/xml provides the data used on the www.g.o pages.

http://www.gentoo.org/proj/en/devrel/roll-call/userinfo.xml looks ok to me.

All done including xsl/dtd/guidexml doc
Thanks for your support & comments.