Edgewall Software

Opened 15 years ago

Last modified 3 years ago

#8146 new defect

Unified diff — at Version 7

Reported by: anonymous Owned by: Christian Boos
Priority: normal Milestone: next-major-releases
Component: version control/changeset view Version: 0.12dev
Severity: normal Keywords: diff javascript bitesized
Cc: sylvain@…, davaeron@… Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Ryan J Ollos)

In the changeset view the unified diff provided by clicking on Unified is different from the one which can be downloaded.

View when selecting Unified in the changeset view:

Index: trunk/setup.py
===================================================================
--- a/trunk/setup.py
+++ b/trunk/setup.py
@@ -2,5 +2,5 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright (C) 2003-2008 Edgewall Software
+# Copyright (C) 2003-2009 Edgewall Software
 # All rights reserved.
 #
@@ -71,5 +71,5 @@
     install_requires = [
         'setuptools>=0.6b1',
-        'Genshi>=0.5'
+        'Genshi>=0.6dev-r960'
     ],
     extras_require = {

Content downloaded as a Unified Diff:

Index: trunk/setup.py
===================================================================
--- trunk/setup.py (revision 7699)
+++ trunk/setup.py (revision 7890)
@@ -2,5 +2,5 @@
 # -*- coding: utf-8 -*-
 #
-# Copyright (C) 2003-2008 Edgewall Software
+# Copyright (C) 2003-2009 Edgewall Software
 # All rights reserved.
 #
@@ -71,5 +71,5 @@
     install_requires = [
         'setuptools>=0.6b1',
-        'Genshi>=0.5'
+        'Genshi>=0.6dev-r960'
     ],
     extras_require = {

here the diff of the two diffs :

  • C:\Users\Absynthe\Downloads\

    old new  
    11Index: trunk/setup.py
    22===================================================================
    3 --- a/trunk/setup.py
    4 +++ b/trunk/setup.py
     3--- trunk/setup.py (revision 7699)
     4+++ trunk/setup.py (revision 7890)
    55@@ -2,5 +2,5 @@
    66 # -*- coding: utf-8 -*-
    77 #

As you can see paths in the index of the first diff are a bit custom.

Change History (8)

comment:1 by Christian Boos, 15 years ago

Milestone: 0.12
Owner: set to Christian Boos

We could effectively remove the a/b prefixes and have the rXXX as suffix, I think.

That would still be a difference, but a very minor one (i.e. the patch command for applying them would be the same).

comment:2 by Sylvain Rabot <sylvain@…>, 15 years ago

Cc: sylvain@… added

by davaeron <davaeron@…>, 15 years ago

Attachment: patch_from_r8172.patch added

comment:3 by davaeron <davaeron@…>, 15 years ago

Priority: normalhigh
Severity: minormajor
Version: 0.11.4rc10.12dev

You broke build with your patch.

To reproduce:

easy_install -U "Genshi==dev"
svn co http://svn.edgewall.org/repos/trac/trunk trac
cd trac
python ./setup.py install

error:

Installed /usr/local/lib/python2.5/site-packages/Trac-0.12dev_r0-py2.5.egg
Processing dependencies for Trac==0.12dev-r0
Searching for Genshi>=0.6dev-r960
Reading http://pypi.python.org/simple/Genshi/
Reading http://genshi.edgewall.org/
Reading http://genshi.edgewall.org/wiki/Download
No local packages or download links found for Genshi>=0.6dev-r960
error: Could not find suitable distribution for Requirement.parse('Genshi>=0.6dev-r960')

Patch included previously. Works for me.

Last edited 9 years ago by Jun Omae (previous) (diff)

comment:4 by davaeron <davaeron@…>, 15 years ago

Cc: davaeron@… added

comment:5 by Christian Boos, 14 years ago

Milestone: 0.12next-minor-0.12.x

comment:6 by Christian Boos, 12 years ago

Keywords: javascript bitesized added
Milestone: next-minor-0.12.xnext-stable-1.0.x

comment:7 by Ryan J Ollos, 9 years ago

Description: modified (diff)
Milestone: next-stable-1.0.xnext-dev-1.1.x
Priority: highnormal
Severity: majornormal
Note: See TracTickets for help on using tickets.