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.

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

trailing slashes in a workdir path #29

Open
remyla opened this issue Jun 11, 2015 · 5 comments
Open

trailing slashes in a workdir path #29

remyla opened this issue Jun 11, 2015 · 5 comments
Assignees

Comments

@remyla
Copy link
Member

remyla commented Jun 11, 2015

users could configure a work dir path like /home/user/work/ (with trailing slash)
if the assets we want to track have an absolute path like /home/user/work/production/chars/rabbit.blend, the system is now proposing to the user to track the file using the path production/chars/rabbit.blend but we would like this path to remain as /production/chars/rabbit.blend

we would like a system to work even if the user wants / as a local work dir

@Baenshee
Copy link

Wouldn't the use of / as a local workdir be like not using a workdir and prevent the simplified creation of workdirs ?

@remyla
Copy link
Member Author

remyla commented Jun 11, 2015

maybe we could have a switch, not to take a '/' workdir into account if we
still want the UI to ask the user if he wants to declare a new workdir?
leave the '/' for the moment if it is asking a lot of modifications. the
trailing slash support is the most important

Rémy

On Thu, Jun 11, 2015 at 12:21 PM, Mathieu [email protected] wrote:

Wouldn't the use of / as a local workdir be like not using a workdir and
prevent the simplified creation of workdirs ?


Reply to this email directly or view it on GitHub
#29 (comment).

@Baenshee
Copy link

Or have a toggle to enable/disable the use of workdirs ?

@remyla
Copy link
Member Author

remyla commented Jun 11, 2015

well, the switch I spoke about was about adding a condition in the program,
not a switch visible from the UI, if this is what you meant. As I said,
having the same behavior as it is now, but with support for trailing
slashes when they occur, would be the priority. If you think of an
enhancement, please elaborate, I will be happy to hear the idea

On Thu, Jun 11, 2015 at 2:37 PM, Mathieu [email protected] wrote:

Or have a toggle to enable/disable the use of workdirs ?


Reply to this email directly or view it on GitHub
#29 (comment).

@Baenshee
Copy link

The support for trailing slashes has been added in my last pull request. I'm still looking for a way to manage the / workdir

remyla added a commit that referenced this issue Jun 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants