patchidutils - Patches: patch #2775, Perl support for id-utils 3.2d

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #2775: Perl support for id-utils 3.2d

Submitter:  None
Submitted:  Wed 10 Mar 2004 06:06:36 AM UTC
   
 
Category:  None Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  sick_soul Originator Email:  -email is unavailable-
Open/Closed:  Closed

Jump to the original submission

Sun 08 Oct 2006 03:27:39 PM UTC, comment #9: 

Changes seem ok to me, thus I am closing the item.
Should any problem with quotation occur again,
I'll reopen.


Claudio Fontana <sick_soul>
Mon 26 Jun 2006 09:19:39 PM UTC, comment #8: 


The changes applied in CVS are functionally identical
to the last proposed patch by Madhu.

The thing seems to work, but some more testing would be appreciated.


Claudio Fontana <sick_soul>
Wed 03 May 2006 04:29:50 AM UTC, comment #7: 

Hi Claudio,

Thank you for reviewing the code. As per my testing, i din't find use for the double quotes modification.  However, after your comments, I realized it is better to include double quotes also. 

I have also noticed from your changes, my comments are NOT descriptive enough to indicate Perl Documentation [perlpod] can be supported by mkid. So, I have reviced comments. 

Could you please include the Patch.txt, which was attched here.

Madhu

Madhu Gogulapati <madhug>
Group Member
Sat 29 Apr 2006 10:25:35 AM UTC, comment #6: 

Hi madghug,
I looked at the suggested changes for the idutils perl support;

do you think that the same should apply to double quotes?
See my attached new unified diff.

Thanks,

Claudio

Claudio Fontana <sick_soul>
Tue 18 Apr 2006 04:07:53 PM UTC, comment #5: 

Hi,
I think, I have fixed the single quote issue.

Could you include the following patch <Doc2html.Patch.txt> into the repository?

Rgd,
Madhu

Madhu Gogulapati <madhug>
Group Member
Thu 16 Feb 2006 05:20:31 AM UTC, comment #4: 

Reopened as bugs seem to be present in this scanner.

Try the attached example perl file and run mkid, lid.
A single quote becomes part of some tokens, and it should not
be there:

$ mkid
$ lid | grep "'"
magic'         doc2html.pl
mime'          doc2html.pl
try_html'      doc2html.pl


Claudio Fontana <sick_soul>
Sat 24 Dec 2005 04:02:27 AM UTC, comment #3: 

integrated.

Claudio Fontana <sick_soul>
Fri 09 Dec 2005 11:01:43 PM UTC, comment #2: 

attached an experimental 4.0 release with perl support patch integrated.

Please report any successes/problems.


Claudio Fontana <sick_soul>
Wed 09 Nov 2005 01:16:41 AM UTC, comment #1: 

this patch includes a small unrelated change to the directory-tree walker (libidu/walker.c) that is already in CVS since 1999 (ChangeLog entry: 1999-04-10  Greg McGary).

The patch also seems to contain some commented and some dead code in get_token_perl()

However, it seems good; I'll work on a restyling of the patch to apply cleanly on current CVS. I will keep it as a separate patch though, at least for now.

Claudio Fontana <sick_soul>
Wed 10 Mar 2004 06:06:36 AM UTC, original submission:  


Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #9880:  Patch.txt added by madhug (2KiB - text/plain - Comments modification to be better describe fuctionality and double quotes )
file #9853:  idutils-perl-support-bugfix.udiff added by sick_soul (1KiB - application/octet-stream - a modified version of the fix provided by madhug that also applies to double quotes)
file #9797:  Doc2html.Patch.txt added by madhug (1KiB - text/plain - Fix related to the reported problem on perl scanner)
file #9579:  doc2html.pl added by sick_soul (17KiB - application/octet-stream - An example perl file that causes problems in the perl scanner)
file #7090:  PerlSupport-3-2d.patch added by None (12KiB - text/plain)

 

Depends on the following items: None found

Items that depend on this one: None found

 

CC list is empty

 

There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

 

Follow 16 latest changes.

Date Changed by Updated Field Previous Value => Replaced by
2006-10-08 sick_soul StatusReady For Test Done
    Open/ClosedOpen Closed
2006-05-03 madhug Attached File- Added Patch.txt, #9880
2006-04-29 sick_soul Attached File- Added idutils-perl-support-bugfix.udiff, #9853
2006-04-18 madhug Attached File- Added Doc2html.Patch.txt, #9797
2006-02-16 sick_soul StatusDone Ready For Test
    Open/ClosedClosed Open
2006-02-16 sick_soul Attached File- Added doc2html.pl, #5942
2006-02-16 sick_soul Carbon-CopyRemoved karl -
2005-12-24 sick_soul StatusIn Progress Done
    Open/ClosedOpen Closed
2005-11-09 sick_soul StatusNone In Progress
    Assigned toNone sick_soul
    Carbon-Copy- Added karl
    Carbon-Copy- Added -email is unavailable-
2004-03-10 None Attached File- Added PerlSupport-3-2d.patch, #2888

Back to the top

Powered by Savane 3.13-758e.
Corresponding source code