Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

colordiff: version 2.x #5

Open
daveewart opened this issue Sep 28, 2012 · 0 comments
Open

colordiff: version 2.x #5

daveewart opened this issue Sep 28, 2012 · 0 comments

Comments

@daveewart
Copy link
Contributor

(I'm continuing applying patches to the colordiff 1.x series - i.e. the 'backport' branch as presented here on GitHub - and distributing those changes via Debian and via direct *.tar.gz download. I'll author new changes to that branch too, as appropriate.)

However, thinking about whether the colordiff 2.x rewrite has the potential to proceed further, I thought it would be useful to agree on a good approach for what would be suitable, development-wise.

Kirk: I've no idea whether you're still interested in doing this or not, because you've never responded to my email or comments on the subject. If you are interested, however, then the remainder of this post might be helpful.

Bearing in mind that the primary distribution route for colordiff is via Debian, it makes sense to treat the state of Debian Unstable as the environment for which new versions of colordiff should be designed. I refer to Debian Unstable as "Sid" below, which is the standard codename for that branch of Debian. That is:

  • Features which rely on a newer Perl version than available in Sid should not be included in colordiff
  • Perl modules which are available "out of the box" in Sid's Perl distribution or which are already Debian-packaged in Sid can be included in colordiff without issue, because they are "guaranteed" to be available in the destination environment.
  • Perl modules which are not natively Debian-packaged or not included in the base Perl install for Sid should not be used.
  • SImilarly, the version of such modules as available in Sid should be considered: features available in module versions more recent than Sid should not be used.
  • To improve cross-distribution compatibility for colordiff outside the Debian family, however, I believe modules should be used sparingly. For minor added functionality, or code simplification, modules should only be used in place of native code if the gain is significant. Any requirement for additional modules will reduce the use of colordiff, since it will become more troublesome to install or to package.

Further info:

  • The package and module contents of Sid are a moving target, always getting newer. The versions of specific packages and modules can be found by searching the "unstable" distribution via http://www.debian.org/distrib/packages
  • Specifically, the current version of Perl in Sid can be found at http://packages.debian.org/sid/perl - it's currently 5.14.2 at the time I write this.
Lekensteyn referenced this issue in Lekensteyn/colordiff Feb 9, 2013
Merging ymattw's code with a few to applying further modifications to ensure that existing behaviour is not modified for existing users.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant