Benoît Person <benoit.person@xxxxxxxxxx> writes: >> Do we want to add that ':' unconditionally? Could GITPERLLIB be >> empty? > > For now, GITPERLLIB is only used in that kind of statements: > use lib (split(/:/, $ENV{GITPERLLIB} || ... )); > > The trailing ':' does not really matter, split will ignore it. That may be true with the current use, but "For now" leaves funny taste, doesn't it? It is not too much trouble to fix by writing GITPERLLIB='@@BUILD_DIR@@/perl/blib/lib'"${GITPERLLIB:+:$GITPERLLIB}" and we will not have to be worried about future changes breaking today's assumption. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html