Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / lazarus / 2011-08

Fixes for lazarus-ccr/cmdline

7 messages · 4 people · –

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

Zaher Dirkey

The units in this package marked as binary files, so i cant make patches for it, would you please change it back to text.

There is simple fix in lazarus-ccr/cmdline/ucmdbox.pas

Fix access read only property
line 1293
Inc(InputPos,UTF8Length(s));
change
Inc(FInputPos,UTF8Length(s));

Thanks in advance

Zaher Dirkey
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus.freepascal.org/pipermail/lazarus/attachments/20110831/c98e16e4/attachment.html>

Reply by email

Vincent Snijders
to Zaher Dirkey

2011/8/31 Zaher Dirkey :

Show quoted text 13 lines
The units in this package marked as binary files, so i cant make patches for it, would you please change it back to text. There is simple fix in lazarus-ccr/cmdline/ucmdbox.pas Fix access read only property line 1293 Inc(InputPos,UTF8Length(s)); change Inc(FInputPos,UTF8Length(s)); Thanks in advance

I took the author's email address from the wiki and forwarded this message. He answered:

Show quoted text 6 lines
I took the 0.5.3 which compiles here and replaced the svn version with it. Let me know if this worked. I don't understand this about the binary files. My "Tortoise SVN" lists all files as plain/text. A Tortoise bug?

Vincent

Reply by email

Zaher Dirkey
to Vincent Snijders

On Wed, Aug 31, 2011 at 5:45 PM, Vincent Snijders <

Show quoted text 32 lines
wrote: 2011/8/31 Zaher Dirkey : > The units in this package marked as binary files, so i cant make patches for > it, would you please change it back to text. > > There is simple fix in lazarus-ccr/cmdline/ucmdbox.pas > > Fix access read only property > line 1293 > Inc(InputPos,UTF8Length(s)); > change > Inc(FInputPos,UTF8Length(s)); > > Thanks in advance > I took the author's email address from the wiki and forwarded this message. He answered: > I took the 0.5.3 which compiles here and replaced the svn version with it. > Let me know if this worked. > I don't understand this about the binary files. My "Tortoise SVN" lists all > files > as plain/text. A Tortoise bug? > Vincent

The fixes compiled now, but still have problem when creating patch

Index: ucmdbox.pas
=================================================================== Cannot display: file marked as a binary type.
svn:mime-type = plain/text

Zaher Dirkey
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lazarus.freepascal.org/pipermail/lazarus/attachments/20110831/35c17faf/attachment.html>

Reply by email

Vincent Snijders
to Zaher Dirkey

2011/8/31 Zaher Dirkey :

Show quoted text 7 lines
The fixes compiled now, but still have problem when creating patch Index: ucmdbox.pas =================================================================== Cannot display: file marked as a binary type. svn:mime-type = plain/text

I have no idea, why it is marked as binary, In svn it is not marked as such.

Vincent

Reply by email

Alexander Klenin
to Zaher Dirkey

2011/9/1 Zaher Dirkey :

True, just need to remove this property from all files and commit it. svn:mime-type = plain/text

This is because the correct spelling is "text/plain"

Alexander S. Klenin

Reply by email