I might say that any time an OS (or at least POSIX) says that something is the same, cdist should treat it the same way, so for your example, the ending / would be ignored.

That's just what makes the most sense to me (and since the standard says X is the case, it makes understanding easier for everyone).

Thanks,
Jake Guffey
Network Security Engineer

eProtex
Network medical device security

5451 Lakeview Parkway S Drive
Indianapolis, Indiana 46268, USA
Mobile: 317-220-7100
jake.guffey@eprotex.com
www.eprotex.com

On Mar 11, 2014, at 4:15 PM, Steven Armstrong <notifications@github.com> wrote:

> In cdist/conf/type/__link/explorer/state:
> 
> > @@ -34,8 +34,12 @@ case "$type" in
> >     symbolic)
> >        cd "$destination_dir"
> >        source_is=$(ls -l "$destination" | sed 's/.*-> //g')
> > -      if [ -h "$destination" -a "$source_is" = "$source" ]; then
> > -         echo present
> > +      if [ -h "$destination" ]; then
> > +         if [ "$source_is" = "$source" ]; then
> RFC: should we ignore trailing slashes when comparing?
> e.g. we currently treat the following as not the same, but they are
> $ ls -l target
> lrwxrwxrwx 1 root root 6 Mar 11 21:02 target -> source
> $ ls -l target
> lrwxrwxrwx 1 sar sar 7 Mar 11 21:10 target -> source/
> 
> —
> Reply to this email directly or view it on GitHub.
>