From b491ffccae20927d60d0dcd972dc8f77394ef2a1 Mon Sep 17 00:00:00 2001
From: mudler <mudler@sabayon.org>
Date: Thu, 21 Apr 2016 21:30:09 +0200
Subject: [PATCH] [builder] check if the content is already there before
 appending content to file

---
 builder | 25 ++++++++++++++++++-------
 1 file changed, 18 insertions(+), 7 deletions(-)

diff --git a/builder b/builder
index c5d94cb..cdb53af 100755
--- a/builder
+++ b/builder
@@ -57,11 +57,20 @@ sub safe_call {
     exit($return) if ($rt);
 }
 
-sub append_to_file(){
-  my ($file_name,$line) = @_;
-  open(my $fh, '>>', $file_name) or die "Could not open file '$filename' $!";
-  print $fh $line."\n";
-  close $fh;
+sub append_to_file {
+    my ( $file_name, $package ) = @_;
+
+    # Check that the package was not already there
+    open my $fh_ro, '<:encoding(UTF-8)', $file_name or die;
+    while ( my $line = <$fh_ro> ) {
+        return if $line eq $package . "\n" ;
+    }
+    close $fh_ro;
+
+    open( my $fh_a, '>>', $file_name )
+      or die "Could not open file '$filename' $!";
+    print $fh_a $package . "\n";
+    close $fh_a;
 }
 
 # Input: package, depth, and atom. Package: sys-fs/foobarfs, Depth: 1 (depth of the package tree) , Atom: 1/0 (enable disable atom output)
@@ -274,12 +283,14 @@ if ($use_equo) {
 
     # best effort mask
     if ($equo_masks) {
-        append_to_file("/etc/entropy/packages/package.mask", $_) for ( split( / /, $equo_masks ) );
+        append_to_file ( "/etc/entropy/packages/package.mask", $_ )
+          for ( split( / /, $equo_masks ) );
     }
 
     # best effort unmask
     if ($equo_unmasks) {
-        append_to_file("/etc/entropy/packages/package.unmask", $_) for ( split( / /, $equo_unmasks ) );
+        append_to_file ( "/etc/entropy/packages/package.unmask", $_ )
+          for ( split( / /, $equo_unmasks ) );
     }
 
     my @packages_deps;