Differential D11629 Diff 31706 kconf_update/gwenview-imageview-alphabackgroundmode-update.pl

Changeset View

Standalone View

1		#!/usr/bin/env perl
2
	ngrahamUnsubmitted Not Done Is there ever going to be an environment where this could run that doesn't have Perl? I'm not one of those Perl haters, but it seems like this could be done more easily and in a more guaranteed-bulletproof manner with `sed` in a shell script instead. ngraham: Is there ever going to be an environment where this could run that doesn't have Perl? I'm not…
	rkflxUnsubmitted Not Done The docs say the following on that topic: stick with interpreted scripts like shell or perl. To make your scripts compatible with Windows, you should not use shell scripts. Also, lots of similar kconf_update scripts are written in Perl, so I guess this is fine. rkflx: [The docs](https://techbase.kde.org/Development/Tools/Using_kconf_update) say the following on…
	huoniAuthorUnsubmitted Not Done Thanks for answering for me :) Those are indeed the reasons I chose Perl. huoni: Thanks for answering for me :) Those are indeed the reasons I chose Perl.
	rkflxUnsubmitted Done This path does not exist on some systems I tested. Konversation uses `/usr/bin/perl`, but I guess the "official" way would be `#!/usr/bin/env perl`. rkflx: This path does not exist on some systems I tested. Konversation uses `/usr/bin/perl`, but I…
3		use strict;
4
5		while (<STDIN>) {
6		s/RasterImageView::AlphaBackgroundSolid/AbstractImageView::AlphaBackgroundSolid/;
7		print $_;
8		}