Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#22329 closed defect (fixed)

subversion diff is printing some %ld

Reported by: hibou@… Owned by: danielluke (Daniel J. Luke)
Priority: Normal Milestone:
Component: ports Version: 1.8.1
Keywords: Cc: landonf (Landon Fuller), rmsfisher@…, ryandesign (Ryan Carsten Schmidt), dackze+macports@…, Ludwig.Huegelschaefer@…, fg@…
Port: subversion

Description (last modified by blb@…)

I have upgraded from leopard to snow leopard, and reinstalled every macport. The subversion I installed is 1.6.5. Since the upgrade the svn diff output is incorrect as the line numbers are %ld:

Index: CHANGES.txt
===================================================================
--- CHANGES.txt	(révision 831578)
+++ CHANGES.txt	(copie de travail)
@@ -%ld,%ld +%ld,%ld @@
 =====================================
 - IMPROVEMENT: Trace a message when a property file referenced from the settings doesn't exixts (IVY-1074)
 - IMPROVEMENT: use defaultconf in combination with defaultconfmapping (IVY-1135) (thanks to Jon Schneider)
+- IMPROVEMENT: Add support for transitive retrieval of source and javadoc artifacts (IVY-1003)
 
 - FIX: <override> ignored in dynamic resolve mode (IVY-1131)
 - FIX: IvyNodeEviction.cleanEvicted has bad comparison logic (IVY-1118)

After a quick search, I have found that the maintainer of this port seems to be aware of that issue while testing subversion 1.6.6: http://subversion.tigris.org/ds/viewMessage.do?dsForumId=1065&dsMessageId=2410378

Hopefully the subversion shipped with xcode tools is a 1.6.2 so I can use it instead.

Change History (20)

comment:1 Changed 14 years ago by blb@…

Description: modified (diff)
Keywords: subversion removed
Owner: changed from macports-tickets@… to dluke@…

comment:2 Changed 14 years ago by jmroot (Joshua Root)

1.6.5 doesn't do this for me. Perhaps the bug only affects some locales?

comment:3 Changed 14 years ago by danielluke (Daniel J. Luke)

Owner: changed from dluke@… to dluke@…
Status: newassigned

I did not see this at all with 1.6.5, but only with 1.6.6.

Perhaps the problem is actually with apr or apr-util?

Do you have either installed as universal binaries?

comment:4 Changed 14 years ago by hibou@…

Here is my locale:

$ locale
LANG="fr_FR.UTF-8"
LC_COLLATE="fr_FR.UTF-8"
LC_CTYPE="fr_FR.UTF-8"
LC_MESSAGES="fr_FR.UTF-8"
LC_MONETARY="fr_FR.UTF-8"
LC_NUMERIC="fr_FR.UTF-8"
LC_TIME="fr_FR.UTF-8"
LC_ALL=

I tried with this:

$ locale                 
LANG="en_US.UTF-8"
LC_COLLATE="en_US.UTF-8"
LC_CTYPE="en_US.UTF-8"
LC_MESSAGES="en_US.UTF-8"
LC_MONETARY="en_US.UTF-8"
LC_NUMERIC="en_US.UTF-8"
LC_TIME="en_US.UTF-8"
LC_ALL=

=> same effect, some %ld in the diff

About apr, I have:

# port installed apr
The following ports are currently installed:
  apr @1.3.9_0 (active)
  apr-util @1.3.9_0 (active)

I tried +universal:

# port installed apr apr-util
The following ports are currently installed:
  apr @1.3.9_0
  apr @1.3.9_0+universal (active)
  apr-util @1.3.9_0
  apr-util @1.3.9_0+universal (active)

=> still some %ld

comment:5 Changed 14 years ago by hibou@…

Ha, some progress here.

+universal apr still active, I rebuild subversion with some new variants:

# port install subversion@1.6.5_0 +tools +bash_completion 
--->  Computing dependencies for subversion
--->  Fetching subversion
--->  Verifying checksum(s) for subversion
--->  Extracting subversion
--->  Applying patches to subversion
--->  Configuring subversion
--->  Building subversion
--->  Staging subversion into destroot
--->  Installing subversion @1.6.6_0+bash_completion+tools
--->  Deactivating subversion @1.6.5_0
--->  Activating subversion @1.6.6_0+bash_completion+tools
--->  Cleaning subversion

=> %ld are gone, there are real numbers now. And with my french locale.

And the above log is correct, I have 1.6.6 in a local repo, based on the 1.6.5 with just the checksums changed. I was expecting to build the 1.6.5 but I may have wrongly specify it on the command line, I am still quite new to mac ports.

I will work with that version of svn for now. If you want me to test other things, ping me.

comment:6 Changed 14 years ago by danielluke (Daniel J. Luke)

Resolution: worksforme
Status: assignedclosed

comment:7 Changed 14 years ago by landonf (Landon Fuller)

Resolution: worksforme
Status: closedreopened

Reopening -- I'm still seeing this behavior locally:

@@ -%ld,%ld +%ld,%ld @@

comment:8 Changed 14 years ago by landonf (Landon Fuller)

Cc: landonf@… added

Cc Me!

comment:9 Changed 14 years ago by danielluke (Daniel J. Luke)

So, to recap:

Both you and I see this with 1.6.6, but you are also seeing it with 1.6.5 (or were you seeing it with your copy of 1.6.6 that you thought was 1.6.5)?

I haven't been able to replicate it with 1.6.5 (and I haven't had time to track down what might be causing it with 1.6.6 yet).

comment:10 Changed 14 years ago by landonf (Landon Fuller)

I'm seeing the behavior on a fresh installation of 1.6.5 on a newly installed Snow Leopard system, as well as all the other Snow Leopard machines here.

comment:11 in reply to:  10 Changed 14 years ago by danielluke (Daniel J. Luke)

Replying to landonf@…:

I'm seeing the behavior on a fresh installation of 1.6.5 on a newly installed Snow Leopard system, as well as all the other Snow Leopard machines here.

+universal or not?

comment:12 Changed 14 years ago by landonf (Landon Fuller)

Seen with both +universal and x86-64. If I recall correctly, I also tested the x86-32 binary when I previously built +universal, and it behaved the same.

comment:13 Changed 14 years ago by jmroot (Joshua Root)

My 1.6.5 is built for x86_64 only on 10.6 and works correctly, so the problem can't be (directly) related to architecture.

comment:14 Changed 14 years ago by rmsfisher@…

Cc: rmsfisher@… added

Cc Me!

comment:15 Changed 14 years ago by ryandesign (Ryan Carsten Schmidt)

Cc: ryandesign@… added

Has duplicates #22565 and #22700.

comment:16 Changed 14 years ago by dackze+macports@…

Cc: dackze+macports@… added

Cc Me!

comment:17 Changed 14 years ago by Ludwig.Huegelschaefer@…

Cc: Ludwig.Huegelschaefer@… added

Cc Me!

comment:18 Changed 14 years ago by fg@…

Cc: fg@… added

Cc Me!

comment:19 Changed 14 years ago by danielluke (Daniel J. Luke)

Resolution: fixed
Status: reopenedclosed

This should be fixed in r62330 (see also #22032) I'll have newer subversion committed shortly (passes all tests for me).

comment:20 Changed 14 years ago by Ludwig.Huegelschaefer@…

Thank you very much! Now also "svnadmin load" works without segfaulting!

Note: See TracTickets for help on using tickets.