Page MenuHomeFreeBSD
Differential D21969

net-mgmt/netdata: Update to 1.18.1
AbandonedPublic
Actions

Authored by diizzy on Oct 10 2019, 1:22 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 24, 7:31 PM
Unknown Object (File)
Fri, Mar 14, 7:34 PM
Unknown Object (File)
Feb 25 2025, 9:29 PM
Unknown Object (File)
Feb 25 2025, 11:37 AM
Unknown Object (File)
Feb 25 2025, 1:31 AM
Unknown Object (File)
Feb 24 2025, 6:22 AM
Unknown Object (File)
Feb 23 2025, 11:10 PM
Unknown Object (File)
Feb 23 2025, 10:05 PM
View All Files
Subscribers
araujo
mat
samm

Details

Reviewers
mmokhi
Summary

Update netdata to 1.18.1
Add option for LTO (enabled by default in older versions)
Update configuration sample and remove irrelevant settings for initial setup

Test Plan

Compile and Runtime tested on AMD64, FreeBSD 13.0-CURRENT r352230

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

diizzy created this revision.Oct 10 2019, 1:22 PM
Herald added a subscriber: mat. · View Herald TranscriptOct 10 2019, 1:22 PM
diizzy updated this revision to Diff 63120.Oct 10 2019, 1:44 PM

Remove USE_LDCONFIG
Refresh patches

diizzy added a comment.Oct 10 2019, 1:46 PM

Poudriere logs{F5549381}

netdata-1.18.0.portlint.log.txt502 BDownload

diizzy updated this revision to Diff 63121.Oct 10 2019, 1:59 PM

Revert changes to files/patch-configure.ac

diizzy updated this revision to Diff 63127.Oct 10 2019, 4:08 PM

Upstream changed release tag

diizzy updated this revision to Diff 63499.Oct 20 2019, 6:48 PM
diizzy retitled this revision from net-mgmt/netdata: Update to 1.18.0 to net-mgmt/netdata: Update to 1.18.1.
diizzy edited the summary of this revision. (Show Details)

Update to 1.18.1

samm added a subscriber: samm.Oct 21 2019, 8:37 AM

Hi there, i already have it here:

https://reviews.freebsd.org/D22045

We need to wait for maintainer (or timeout) to merge it.

diizzy mentioned this in D22045: net-mgmt/netdata: upgrade to 1.18.1.Oct 21 2019, 9:35 AM
diizzy updated this revision to Diff 63512.Oct 21 2019, 11:01 AM

Remove GH_ACCOUNT variable
Update pkg-descr with correct URL
These changes are taken from D22045 made by Alex Samorukov

araujo added a subscriber: araujo.Oct 23 2019, 8:38 AM

Can we abandon this review in favor of the another one?

diizzy added a comment.Oct 23 2019, 8:45 AM

At least fix the issues reported in that case?

araujo added a comment.Oct 23 2019, 8:51 AM
In D21969#483371, @daniel.engberg.lists_pyret.net wrote:

At least fix the issues reported in that case?

Sure, why not? My suggestion to abandon it here is to focus in a single place and you will receive the credits too for the changes. It will make easy the review as well as makes the maintainers life easier.

diizzy abandoned this revision.Oct 23 2019, 9:02 AM

Sure, but I think it's a discouraging way to handle submissions in terms of chronological order.

araujo added a comment.Oct 23 2019, 9:07 AM
In D21969#483377, @daniel.engberg.lists_pyret.net wrote:

Sure, but I think it's a discouraging way to handle submissions in terms of chronological order.

It is not my intention to discouraging you to keep submitting patches! Please, sync with @samm the changes you would like to see there and were not applied yet. You will get the credits for this update, no worries.

diizzy mentioned this in D22615: net-mgmt/netdata: Update to 1.19.0.Nov 30 2019, 10:51 PM

Revision Contents
Changeset List

PathSize
net-mgmt/
netdata/
10 lines
6 lines
files/
3396 lines
13 lines
7 lines

Diff 63127

View Options

net-mgmt/netdata/Makefile

Show All 14 Lines
# #
# This replaces the old pattern: # This replaces the old pattern:
# @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi # @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi
# etc/pkgtools.conf.sample # etc/pkgtools.conf.sample
# @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf # @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf
actions: [file(1)] actions: [file(1)]
arguments: true arguments: true
post-install: <<EOD post-install-lua: <<EOS
case "%1" in sample_file = pkg.prefixed_path("%1", "%D")
/*) sample_file="%1" ;; _,n = string.gsub("%@", "%S+","")
*) sample_file="%D/%1" ;; if n == 2 then
esac target_file = pkg.prefixed_path("%1", "%D")
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if ! [ -f "${target_file}" ]; then
/bin/cp -p "${sample_file}" "${target_file}"
fi
EOD
pre-deinstall: <<EOD
case "%1" in
/*) sample_file="%1" ;;
*) sample_file="%D/%1" ;;
esac
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
set -- %@
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if cmp -s "${target_file}" "${sample_file}"; then
rm -f "${target_file}"
else else
echo "You may need to manually remove ${target_file} if it is no longer needed." target_file = string.gsub(sample_file,'%.sample$', "")
fi end
EOD outh = io.open(target_file, "r")
matUnsubmitted
Done
Inline Actions

There is so much magic going on here.

What is %@ ? What is %S+ ? I looked at pkg-lua-script(5) and it does not appear in it.

mat: There is so much magic going on here. What is `%@` ? What is `%S+` ? I looked at pkg-lua…
manuUnsubmitted
Done
Inline Actions

I copied the gsub from the old review but a simple test on %# will be enough. I'll update the code.

manu: I copied the gsub from the old review but a simple test on %# will be enough. I'll update the…
if outh ~= nil then
io.close(outh)
return
end
outh = io.open(target_file, "w+")
for line in io.lines(sample_file) do
outh:write(line, "\n")
end
matUnsubmitted
Done
Inline Actions

This is not what the script was doing before.

The script copies the .sample to the destination file if the destination file does not exist. There is absolutely no comparison involved. There should be a pkg.fileexists or something.

mat: This is not what the script was doing before. The script copies the .sample to the destination…
manuUnsubmitted
Done
Inline Actions

This is exactly what it does too.
Sure there is a "useless" cmp, I can see to add a fileexist (or use of the lua os. function maybe)

manu: This is exactly what it does too. Sure there is a "useless" cmp, I can see to add a fileexist…
io.close(outh)
EOS
pre-deinstall-lua: <<EOS
sample_file = pkg.prefixed_path("%1")
_,n = string.gsub("%@", "%S+","")
if n == 2 then
target_file = pkg.prefixed_path("%1")
else
target_file = string.gsub(sample_file, '%.sample$', "")
end
outh = io.open(target_file, "r")
inh = io.open(sample_file, "r")
sizeout = outh:seek("end")
sizein = inh:seek("end")
if sizeout ~= sizein then
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
return
end
inh:seek("set", 0)
outh:seek("set", 0)
linein = inh:read("l")
lineout = outh:read("l")
while (linein == lineout and linein ~= nil and lineout ~= nil) do
linein = inh:read("l")
lineout = outh:read("l")
end
io.close(inh)
io.close(outh)
if linein == lineout then
os.remove(target_file)
else
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
end
EOS
View Options

net-mgmt/netdata/distinfo

Show All 14 Lines
# #
# This replaces the old pattern: # This replaces the old pattern:
# @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi # @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi
# etc/pkgtools.conf.sample # etc/pkgtools.conf.sample
# @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf # @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf
actions: [file(1)] actions: [file(1)]
arguments: true arguments: true
post-install: <<EOD post-install-lua: <<EOS
case "%1" in sample_file = pkg.prefixed_path("%1", "%D")
/*) sample_file="%1" ;; _,n = string.gsub("%@", "%S+","")
*) sample_file="%D/%1" ;; if n == 2 then
esac target_file = pkg.prefixed_path("%1", "%D")
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if ! [ -f "${target_file}" ]; then
/bin/cp -p "${sample_file}" "${target_file}"
fi
EOD
pre-deinstall: <<EOD
case "%1" in
/*) sample_file="%1" ;;
*) sample_file="%D/%1" ;;
esac
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
set -- %@
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if cmp -s "${target_file}" "${sample_file}"; then
rm -f "${target_file}"
else else
echo "You may need to manually remove ${target_file} if it is no longer needed." target_file = string.gsub(sample_file,'%.sample$', "")
fi end
EOD outh = io.open(target_file, "r")
matUnsubmitted
Done
Inline Actions

There is so much magic going on here.

What is %@ ? What is %S+ ? I looked at pkg-lua-script(5) and it does not appear in it.

mat: There is so much magic going on here. What is `%@` ? What is `%S+` ? I looked at pkg-lua…
manuUnsubmitted
Done
Inline Actions

I copied the gsub from the old review but a simple test on %# will be enough. I'll update the code.

manu: I copied the gsub from the old review but a simple test on %# will be enough. I'll update the…
if outh ~= nil then
io.close(outh)
return
end
outh = io.open(target_file, "w+")
for line in io.lines(sample_file) do
outh:write(line, "\n")
end
matUnsubmitted
Done
Inline Actions

This is not what the script was doing before.

The script copies the .sample to the destination file if the destination file does not exist. There is absolutely no comparison involved. There should be a pkg.fileexists or something.

mat: This is not what the script was doing before. The script copies the .sample to the destination…
manuUnsubmitted
Done
Inline Actions

This is exactly what it does too.
Sure there is a "useless" cmp, I can see to add a fileexist (or use of the lua os. function maybe)

manu: This is exactly what it does too. Sure there is a "useless" cmp, I can see to add a fileexist…
io.close(outh)
EOS
pre-deinstall-lua: <<EOS
sample_file = pkg.prefixed_path("%1")
_,n = string.gsub("%@", "%S+","")
if n == 2 then
target_file = pkg.prefixed_path("%1")
else
target_file = string.gsub(sample_file, '%.sample$', "")
end
outh = io.open(target_file, "r")
inh = io.open(sample_file, "r")
sizeout = outh:seek("end")
sizein = inh:seek("end")
if sizeout ~= sizein then
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
return
end
inh:seek("set", 0)
outh:seek("set", 0)
linein = inh:read("l")
lineout = outh:read("l")
while (linein == lineout and linein ~= nil and lineout ~= nil) do
linein = inh:read("l")
lineout = outh:read("l")
end
io.close(inh)
io.close(outh)
if linein == lineout then
os.remove(target_file)
else
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
end
EOS
View Options

net-mgmt/netdata/files/netdata.conf.sample.in

Show All 14 Lines
# #
# This replaces the old pattern: # This replaces the old pattern:
# @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi # @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi
# etc/pkgtools.conf.sample # etc/pkgtools.conf.sample
# @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf # @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf
actions: [file(1)] actions: [file(1)]
arguments: true arguments: true
post-install: <<EOD post-install-lua: <<EOS
case "%1" in sample_file = pkg.prefixed_path("%1", "%D")
/*) sample_file="%1" ;; _,n = string.gsub("%@", "%S+","")
*) sample_file="%D/%1" ;; if n == 2 then
esac target_file = pkg.prefixed_path("%1", "%D")
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if ! [ -f "${target_file}" ]; then
/bin/cp -p "${sample_file}" "${target_file}"
fi
EOD
pre-deinstall: <<EOD
case "%1" in
/*) sample_file="%1" ;;
*) sample_file="%D/%1" ;;
esac
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
set -- %@
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if cmp -s "${target_file}" "${sample_file}"; then
rm -f "${target_file}"
else else
echo "You may need to manually remove ${target_file} if it is no longer needed." target_file = string.gsub(sample_file,'%.sample$', "")
fi end
EOD outh = io.open(target_file, "r")
matUnsubmitted
Done
Inline Actions

There is so much magic going on here.

What is %@ ? What is %S+ ? I looked at pkg-lua-script(5) and it does not appear in it.

mat: There is so much magic going on here. What is `%@` ? What is `%S+` ? I looked at pkg-lua…
manuUnsubmitted
Done
Inline Actions

I copied the gsub from the old review but a simple test on %# will be enough. I'll update the code.

manu: I copied the gsub from the old review but a simple test on %# will be enough. I'll update the…
if outh ~= nil then
io.close(outh)
return
end
outh = io.open(target_file, "w+")
for line in io.lines(sample_file) do
outh:write(line, "\n")
end
matUnsubmitted
Done
Inline Actions

This is not what the script was doing before.

The script copies the .sample to the destination file if the destination file does not exist. There is absolutely no comparison involved. There should be a pkg.fileexists or something.

mat: This is not what the script was doing before. The script copies the .sample to the destination…
manuUnsubmitted
Done
Inline Actions

This is exactly what it does too.
Sure there is a "useless" cmp, I can see to add a fileexist (or use of the lua os. function maybe)

manu: This is exactly what it does too. Sure there is a "useless" cmp, I can see to add a fileexist…
io.close(outh)
EOS
pre-deinstall-lua: <<EOS
sample_file = pkg.prefixed_path("%1")
_,n = string.gsub("%@", "%S+","")
if n == 2 then
target_file = pkg.prefixed_path("%1")
else
target_file = string.gsub(sample_file, '%.sample$', "")
end
outh = io.open(target_file, "r")
inh = io.open(sample_file, "r")
sizeout = outh:seek("end")
sizein = inh:seek("end")
if sizeout ~= sizein then
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
return
end
inh:seek("set", 0)
outh:seek("set", 0)
linein = inh:read("l")
lineout = outh:read("l")
while (linein == lineout and linein ~= nil and lineout ~= nil) do
linein = inh:read("l")
lineout = outh:read("l")
end
io.close(inh)
io.close(outh)
if linein == lineout then
os.remove(target_file)
else
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
end
EOS
View Options

net-mgmt/netdata/files/patch-collectors_python.d.plugin_python.d.plugin.in

Show All 14 Lines
# #
# This replaces the old pattern: # This replaces the old pattern:
# @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi # @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi
# etc/pkgtools.conf.sample # etc/pkgtools.conf.sample
# @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf # @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf
actions: [file(1)] actions: [file(1)]
arguments: true arguments: true
post-install: <<EOD post-install-lua: <<EOS
case "%1" in sample_file = pkg.prefixed_path("%1", "%D")
/*) sample_file="%1" ;; _,n = string.gsub("%@", "%S+","")
*) sample_file="%D/%1" ;; if n == 2 then
esac target_file = pkg.prefixed_path("%1", "%D")
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if ! [ -f "${target_file}" ]; then
/bin/cp -p "${sample_file}" "${target_file}"
fi
EOD
pre-deinstall: <<EOD
case "%1" in
/*) sample_file="%1" ;;
*) sample_file="%D/%1" ;;
esac
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
set -- %@
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if cmp -s "${target_file}" "${sample_file}"; then
rm -f "${target_file}"
else else
echo "You may need to manually remove ${target_file} if it is no longer needed." target_file = string.gsub(sample_file,'%.sample$', "")
fi end
EOD outh = io.open(target_file, "r")
matUnsubmitted
Done
Inline Actions

There is so much magic going on here.

What is %@ ? What is %S+ ? I looked at pkg-lua-script(5) and it does not appear in it.

mat: There is so much magic going on here. What is `%@` ? What is `%S+` ? I looked at pkg-lua…
manuUnsubmitted
Done
Inline Actions

I copied the gsub from the old review but a simple test on %# will be enough. I'll update the code.

manu: I copied the gsub from the old review but a simple test on %# will be enough. I'll update the…
if outh ~= nil then
io.close(outh)
return
end
outh = io.open(target_file, "w+")
for line in io.lines(sample_file) do
outh:write(line, "\n")
end
matUnsubmitted
Done
Inline Actions

This is not what the script was doing before.

The script copies the .sample to the destination file if the destination file does not exist. There is absolutely no comparison involved. There should be a pkg.fileexists or something.

mat: This is not what the script was doing before. The script copies the .sample to the destination…
manuUnsubmitted
Done
Inline Actions

This is exactly what it does too.
Sure there is a "useless" cmp, I can see to add a fileexist (or use of the lua os. function maybe)

manu: This is exactly what it does too. Sure there is a "useless" cmp, I can see to add a fileexist…
io.close(outh)
EOS
pre-deinstall-lua: <<EOS
sample_file = pkg.prefixed_path("%1")
_,n = string.gsub("%@", "%S+","")
if n == 2 then
target_file = pkg.prefixed_path("%1")
else
target_file = string.gsub(sample_file, '%.sample$', "")
end
outh = io.open(target_file, "r")
inh = io.open(sample_file, "r")
sizeout = outh:seek("end")
sizein = inh:seek("end")
if sizeout ~= sizein then
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
return
end
inh:seek("set", 0)
outh:seek("set", 0)
linein = inh:read("l")
lineout = outh:read("l")
while (linein == lineout and linein ~= nil and lineout ~= nil) do
linein = inh:read("l")
lineout = outh:read("l")
end
io.close(inh)
io.close(outh)
if linein == lineout then
os.remove(target_file)
else
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
end
EOS
View Options

net-mgmt/netdata/pkg-plist

Show All 14 Lines
# #
# This replaces the old pattern: # This replaces the old pattern:
# @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi # @unexec if cmp -s %D/etc/pkgtools.conf %D/etc/pkgtools.conf.sample; then rm -f %D/etc/pkgtools.conf; fi
# etc/pkgtools.conf.sample # etc/pkgtools.conf.sample
# @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf # @exec [ -f %B/pkgtools.conf ] || cp %B/%f %B/pkgtools.conf
actions: [file(1)] actions: [file(1)]
arguments: true arguments: true
post-install: <<EOD post-install-lua: <<EOS
case "%1" in sample_file = pkg.prefixed_path("%1", "%D")
/*) sample_file="%1" ;; _,n = string.gsub("%@", "%S+","")
*) sample_file="%D/%1" ;; if n == 2 then
esac target_file = pkg.prefixed_path("%1", "%D")
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if ! [ -f "${target_file}" ]; then
/bin/cp -p "${sample_file}" "${target_file}"
fi
EOD
pre-deinstall: <<EOD
case "%1" in
/*) sample_file="%1" ;;
*) sample_file="%D/%1" ;;
esac
target_file="${sample_file%.sample}"
set -- %@
if [ $# -eq 2 ]; then
set -- %@
target_file=${2}
fi
case "${target_file}" in
/*) target_file="${target_file}" ;;
*) target_file="%D/${target_file}" ;;
esac
if cmp -s "${target_file}" "${sample_file}"; then
rm -f "${target_file}"
else else
echo "You may need to manually remove ${target_file} if it is no longer needed." target_file = string.gsub(sample_file,'%.sample$', "")
fi end
EOD outh = io.open(target_file, "r")
matUnsubmitted
Done
Inline Actions

There is so much magic going on here.

What is %@ ? What is %S+ ? I looked at pkg-lua-script(5) and it does not appear in it.

mat: There is so much magic going on here. What is `%@` ? What is `%S+` ? I looked at pkg-lua…
manuUnsubmitted
Done
Inline Actions

I copied the gsub from the old review but a simple test on %# will be enough. I'll update the code.

manu: I copied the gsub from the old review but a simple test on %# will be enough. I'll update the…
if outh ~= nil then
io.close(outh)
return
end
outh = io.open(target_file, "w+")
for line in io.lines(sample_file) do
outh:write(line, "\n")
end
matUnsubmitted
Done
Inline Actions

This is not what the script was doing before.

The script copies the .sample to the destination file if the destination file does not exist. There is absolutely no comparison involved. There should be a pkg.fileexists or something.

mat: This is not what the script was doing before. The script copies the .sample to the destination…
manuUnsubmitted
Done
Inline Actions

This is exactly what it does too.
Sure there is a "useless" cmp, I can see to add a fileexist (or use of the lua os. function maybe)

manu: This is exactly what it does too. Sure there is a "useless" cmp, I can see to add a fileexist…
io.close(outh)
EOS
pre-deinstall-lua: <<EOS
sample_file = pkg.prefixed_path("%1")
_,n = string.gsub("%@", "%S+","")
if n == 2 then
target_file = pkg.prefixed_path("%1")
else
target_file = string.gsub(sample_file, '%.sample$', "")
end
outh = io.open(target_file, "r")
inh = io.open(sample_file, "r")
sizeout = outh:seek("end")
sizein = inh:seek("end")
if sizeout ~= sizein then
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
return
end
inh:seek("set", 0)
outh:seek("set", 0)
linein = inh:read("l")
lineout = outh:read("l")
while (linein == lineout and linein ~= nil and lineout ~= nil) do
linein = inh:read("l")
lineout = outh:read("l")
end
io.close(inh)
io.close(outh)
if linein == lineout then
os.remove(target_file)
else
pkg.print_msg("You may need to manually remove " .. target_file .. " if it is no longer needed.")
end
EOS