Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

!contents -p not showing symbolic link attribute, as documented #541

Closed
ALuckyGuy opened this issue Oct 17, 2014 · 5 comments
Closed

!contents -p not showing symbolic link attribute, as documented #541

ALuckyGuy opened this issue Oct 17, 2014 · 5 comments

Comments

@ALuckyGuy
Copy link

Running both an August 24th and October 6th build, the symbolic link attribute, default '>', is never displayed when a !contents -p option is used. Not sure if it ever worked, and checking the Acceptance tests, I can't see that there are any tests to verify the behavior.

Additionally, I don't see tests for verifying that the global variable PROPERTY_CHARACTERS works, and it doesn't seem to either.
@mwarhaftig
Copy link
Contributor

Hi ALuckyGuy, I recreated the behavior you are seeing with HEAD - let me try to track down the cause.

mwarhaftig added a commit to mwarhaftig/fitnesse that referenced this issue Nov 9, 2014
@mwarhaftig
Issue unclebob#541 - Added SymLinks & PROPERTY_CHARACTERS variable to…
c0f06ef 
… !contents.
@mwarhaftig
Copy link
Contributor

Above pull request fixes both of the !contents issues you identified: missing symbolic link property suffixes & ignored PROPERTY_CHARACTERS variables.

@ALuckyGuy
Copy link
Author

That's awesome... thank you!

amolenaar added a commit that referenced this issue Nov 11, 2014
@amolenaar
Merge pull request #552 from mwarhaftig/master
b5989d7 
Issue #541 - Added SymLinks & PROPERTY_CHARACTERS variable to !contents.
@ALuckyGuy
Copy link
Author

Seems to be working. Thanks again.

@amolenaar
Copy link
Collaborator

Closing issue. A fix has been merged a while back already.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@amolenaar @mwarhaftig @ALuckyGuy