Package: topgit Version: 0.5-1 Followup-For: Bug #513952 Hi Martin,
I finally got around to dive into TopGit with CDBS. And was hurt by this bug. Attached is the cdbs snippet that I wrote for my own use. I prefer declaring that I use TopGit. This has the benefit over Stefanos approach of including patchsys-quilt.mk that if you ever restructure to not need quilt, then we (in principle) need no changes to each package. Please consider shipping the attached CDBS snippet with topgit: 1) install it as /usr/share/cdbs/1/topgit.mk 2) have the binary "topgit" package suggest cdbs Including my proposed snippet does not exactly solve this bug, but it turns it into a less severe one: CDBS use majuscles (capital letters) for variables allowed to be changed and minuscles for "internal" variables (i.e. possibly shared across snippets but not for the packages to redefine). The current need for the topgit.mk snippet to declare PATCHES_DIR is confusing as it might wrongly be seen as an invitation for overriding. ...you should probably have the package suggest quilt as well, but that really is a separate issue - and I am too eager to play some more with TopGit to file a separate bugreport about it :-P Kind regards, - Jonas -- System Information: Debian Release: squeeze/sid APT prefers unstable APT policy: (500, 'unstable') Architecture: amd64 (x86_64) Kernel: Linux 2.6.28-1-amd64 (SMP w/2 CPU cores) Locale: LANG=da_DK.UTF-8, LC_CTYPE=da_DK.UTF-8 (charmap=UTF-8) Shell: /bin/sh linked to /bin/bash Versions of packages topgit depends on: ii git-core 1:1.6.1.3-1 fast, scalable, distributed revisi topgit recommends no packages. topgit suggests no packages. -- no debconf information
# -*- mode: makefile; coding: utf-8 -*- # Copyright © 2009 Jonas Smedegaard <d...@jones.dk> # Description: let Handle patches using TopGit and quilt # # This program is free software; you can redistribute it and/or # modify it under the terms of the GNU General Public License as # published by the Free Software Foundation; either version 2, or (at # your option) any later version. # # This program is distributed in the hope that it will be useful, but # WITHOUT ANY WARRANTY; without even the implied warranty of # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU # General Public License for more details. # # You should have received a copy of the GNU General Public License # along with this program; if not, write to the Free Software # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA # 02111-1307 USA. _cdbs_scripts_path ?= /usr/lib/cdbs _cdbs_rules_path ?= /usr/share/cdbs/1/rules _cdbs_class_path ?= /usr/share/cdbs/1/class ifndef _cdbs_rules_topgit _cdbs_rules_topgit = 1 include $(_cdbs_rules_path)/patchsys-quilt.mk$(_cdbs_makefile_suffix) # Currently TopGit is not used in any policy-specified build targets #CDBS_BUILD_DEPENDS := $(CDBS_BUILD_DEPENDS), topgit # These are hardcoded!! # TODO: convince patchsys-quilt.mk and tg2quilt.mk to use our variables PATCHES_DIR := $(DEB_PATCHDIRS) QUILT_STAMPFN := debian/stamp-patched include /usr/share/topgit/tg2quilt.mk endif