missing implimentation for slack's "files" (#27) about cpm HOT 13 CLOSED

i'm trying to come up with a replacement for cat and tail, and i'm running into a wall. the way i would go about this including coreutils would be something like the following
also i need help just making sure this works

# list all in a text file piped to lsftr after string $1
# usage: filetowrite="/dir/file"; cat $(filetowrite) | lsftr $(filetowrite) "FILES:"
lsftr() {
  i=0
  while read -r line; if [ "$2" != $(STDIN) ]; do
    i=$((i + 1))
  done
  tail +$(i) "$1"
}
}

from cpm.

if i'm overcomplicating this, and there's an easy way to do this, please tell me

from cpm.

What is the high level goal here?

from cpm.

Oh, I see. I think it's best to do something like this:

# come up with a more creative name for this function
filelist() {
  while IFS= read -r line; do
    [ "$line" = "FILE LIST:" ] && break
  done
  while IFS= read -r line; do
    printf "%s\n" "$line"
  done
}

@6gk thoughts?

from cpm.

This can be combined into one loop with a flag variable and another conditional, which is probably clearer.

from cpm.

Sage's impl hurts my brain

I guess yours seems fine, will. However I'm still really confused about what's happening, not helped by not even knowing the file format or idea.

from cpm.

See #27, that's where I just went and found the file in question. I think the screenshot is showing the file/command output to be parsed.

from cpm.

from cpm.

Something like this might work:

slacklist() {
  pkginfo="/var/whatever/$1"
  if [ -f "$pkginfo" ];
    unset found
    while IFS= read -r line; do
      [ "$line" = "FILE LIST:" ] && found=1
      [ "$found" ] && printf "%s\n" "$line"
    done <"$pkginfo"
  else
    # error handling
  fi
}

Call it as slacklist zsh, for example.

from cpm.

Unless there there's weird things going on with whitespace (trailing, leading, multiple mid-word, etc), you shouldn't need to specify IFS=

I almost always do it anyway, just in case for some reason someone has set it to something else in an enclosing scope or outside the script for some reason, that sort of thing.

from cpm.

Sage's impl hurts my brain

i've been overthinking this for months now, so that's a fair opinion

for error handling it should be something of sorts like
(this is in psuedo as i'm away from my cpu atm)

if [ -d /var/pkgthing ]
  if [ -f "/var/pkgthing/$1" ]
    pem "something happened"
   else pem "not in db, try updating"
  else pem "no db, are you using slackware?"

from cpm.

also, that var directory is /var/log/packages/PACKAGE

from cpm.

i've put it all together at https://github.com/foxsouns/cpm/tree/slackfiles, i guess ill pr and let reviews happen

from cpm.