Bug#339826: homepage pseudofield

2006-01-01 Thread Justin Pryzby
On Fri, Dec 30, 2005 at 10:22:21AM +0100, Andreas Barth wrote:
 tags 339826 - pending
 thanks
 
 * Justin Pryzby ([EMAIL PROTECTED]) [051230 03:07]:
  It now has:
  
  Please make sure that this line matches the regular expression
  /^ Homepage: [^ ]*$/, as this allows packages.debian.org to parse
  it correctly.
  
  But this is wrong IMO, and should be /^  Homepage: [^ ]*$/.
  I just checked the CVS and this appears to be a problem with HTML
  conversion only.
 
 Oh no, not another HTML-issue again.
Sorry :/

What about ... extended regular expression: m/^ {2}Homepage: [^ ]*$/

As much as I hate to work around a bug (which I guess this is), this
is even more visually unambigious.

-- 
Clear skies,
Justin


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#339826: homepage pseudofield

2005-12-30 Thread Andreas Barth
tags 339826 - pending
thanks

* Justin Pryzby ([EMAIL PROTECTED]) [051230 03:07]:
 It now has:
 
 Please make sure that this line matches the regular expression
 /^ Homepage: [^ ]*$/, as this allows packages.debian.org to parse
 it correctly.
 
 But this is wrong IMO, and should be /^  Homepage: [^ ]*$/.
 I just checked the CVS and this appears to be a problem with HTML
 conversion only.

Oh no, not another HTML-issue again.

 But, I'm including a patch which reworks this section, and

Ok, I'll apply this patch next time I'm going over bugs.
Thanks.


Cheers,
Andi
-- 
  http://home.arcor.de/andreas-barth/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]



Bug#339826: homepage pseudofield

2005-12-29 Thread Justin Pryzby
Hello Andreas,

I'm still not happy with DevRef WRT the 'homepage' field.

#339826: Please discuss Homepage padding more fully

It now has:

Please make sure that this line matches the regular expression
/^ Homepage: [^ ]*$/, as this allows packages.debian.org to parse
it correctly.

But this is wrong IMO, and should be /^  Homepage: [^ ]*$/.
I just checked the CVS and this appears to be a problem with HTML
conversion only.

I don't know how to fix that problem.

But, I'm including a patch which reworks this section, and

  . Emphasises that this pseudofield is special, and handled
specially;

  . Makes it easier to see the extra indentation;

  . Stops discouraging people from using the pseudofield, because it
will eventually be a real field

  . Gives a fuller example

  . Fixes a grammar problem

-- 
Clear skies,
Justin
--- /tmp/developers-reference.sgml  2005-12-29 20:39:54.0 -0500
+++ /tmp/developers-reference.sgml-new  2005-12-29 20:40:07.0 -0500
@@ -3833,26 +3833,30 @@
 We recommend that you add the URL for the package's home page to the
 package description in filedebian/control/file.  This information
 should be added at the
-end of description, using the following format:
+end of the long description, using a special format recognized by
+filepackages.debian.org/file.  You can see an example of how the
+homepage will be displayed at
+url id=url-eg-desc-upstream-info;.
+
+A package using this field will have have a Description such as:
 
 example .
+Description: foo
+ bar
+ .
   Homepage: http://some-project.some-place.org//example
 
-Note the spaces prepending the line, which serves to break the lines
-correctly.  To see an example of how this displays, see url
-id=url-eg-desc-upstream-info;.
+Note the spaces prepending the last line.  Packages using the Homepage
+pseudofield will have filedebian/control/file files which match
+the regular expression tt/^  Homepage: [^ ]*$//tt.
+
   p
 If there is no home page for the software, this should naturally be
 left out.
   p
 Note that we expect this field will eventually be replaced by a proper
 filedebian/control/file field understood by prgndpkg/prgn and
-ttpackages-host;/tt.  If you don't want to bother migrating the
-home page from the description to this field, you should probably wait
-until that is available.
- Please make sure that this line matches the regular expression
- tt/^  Homepage: [^ ]*$//tt,
- as this allows filepackages.debian.org/file to parse it correctly./p
+ttpackages-host;/tt./p
 /sect1
   /sect