526192 – app-portage/g-cpan-0.16.5 interprets tabs in make.conf as letter 't'

Bug 526192 - app-portage/g-cpan-0.16.5 interprets tabs in make.conf as letter 't'

Summary: app-portage/g-cpan-0.16.5 interprets tabs in make.conf as letter 't'

Status:	RESOLVED FIXED

Alias:	None

Product:	Portage Development
Classification:	Unclassified
Component:	Tools (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Robin Johnson

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2014-10-21 08:15 UTC by Martin von Gagern
Modified:	2014-12-19 19:45 UTC (History)
CC List:	1 user (show)

See Also:	https://github.com/gentoo-perl/g-cpan/issues/16
Package list:
Runtime testing required:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Martin von Gagern 2014-10-21 08:15:55 UTC

Even after fixing the warnings from bug 526188, g-cpan reports

* DEFINED OVERLAYS DON'T EXIST!

This is because the PORTDIR_OVERLAY variable in my make.conf contains tabs. "set" reports these as $'…\t…' and strip_env from Gentoo::Portage (as installed by g-cpan) removes the '\' but keeps the 't'. So I guess we should add \\t to the \\n handling in there.

- s{\\n}{ }gxms
+ s{\\n|\\t}{ }gxms

Comment 1 Martin von Gagern 2014-10-21 09:54:02 UTC

Filed https://github.com/gentoo-perl/g-cpan/pull/16 for my suggested change.

Comment 2 Andreas K. Hüttel archtester

2014-12-13 23:41:18 UTC

Merged into git master. Leaving the bug open until a package is released.

Comment 3 Andreas K. Hüttel archtester

2014-12-19 19:45:15 UTC

(In reply to Andreas K. Hüttel from comment #2)
> Merged into git master. Leaving the bug open until a package is released.

Fix included in 0.16.6