[buildd-tools-devel] [PATCH] Output the contents of debian/files w/o a box

Max Thoursie max at spotify.com
Wed Jan 25 10:20:48 UTC 2012


I don't think drawing a box around the contents of a file is appropriate
since:

* Boxes are already used for headings which could be confusing if the
  file only has one line.
* It makes cut'n'pasting of the contents harder.
---
 lib/Sbuild/Build.pm |    6 +-----
 1 files changed, 1 insertions(+), 5 deletions(-)

diff --git a/lib/Sbuild/Build.pm b/lib/Sbuild/Build.pm
index 8209389..ae636d7 100644
--- a/lib/Sbuild/Build.pm
+++ b/lib/Sbuild/Build.pm
@@ -1384,17 +1384,13 @@ sub build {
 	open( FILES, "<$dscdir/debian/files" );
 	chomp( @lines = <FILES> );
 	close( FILES );
-	@lines = map { my $ind = 76-length($_);
-		       $ind = 0 if $ind < 0;
-		       "│ $_".(" " x $ind). " │\n"; } @lines;
 
 	$self->log_warning("After unpacking, there exists a file debian/files with the contents:\n");
 
-	$self->log('┌', '─'x78, '┐', "\n");
 	foreach (@lines) {
 	    $self->log($_);
 	}
-	$self->log('└', '─'x78, '┘', "\n");
+	$self->log("\n");
 
 	$self->log_info("This should be reported as a bug.\n");
 	$self->log_info("The file has been removed to avoid dpkg-genchanges errors.\n");
-- 
1.7.5.4




More information about the Buildd-tools-devel mailing list