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

logging can be improved to log at all times #63

Open
ksobon opened this issue Jan 18, 2018 · 3 comments
Open

logging can be improved to log at all times #63

ksobon opened this issue Jan 18, 2018 · 3 comments
Assignees

Comments

@ksobon
Copy link
Contributor

ksobon commented Jan 18, 2018

currently logs get written at the end of the revit session when revit is closing. It would be best to capture them at all times. NLog is one library that can be used for that.

@ksobon ksobon self-assigned this Jan 18, 2018
@ksobon
Copy link
Contributor Author

ksobon commented Feb 12, 2018

@dbaldacchino if I wanted to improve logging ability of this particular tool, I could use a library called NLog. It does however require that we install a DLL in the same folder that Revit is installed in. That means that we need ability to place a DLL in C:\ProgramFiles...any issues with that?

@dbaldacchino
Copy link
Contributor

We can place as part of the install/update script moving forward. If this is something we need to add right now to all installs, we could also do a script and put in Group Policy to execute once. Let me know where I can find the file (I assume it would be the same for all versions of Revit?)

@ksobon
Copy link
Contributor Author

ksobon commented Feb 12, 2018

Yeah, it would be the same DLL. So the one thing that bothers me is that it would now be decoupled from our standard deployment for plug-ins. I don't like that very much. Any way that we can think of to keep this under the "revit plugins deployments" umbrella?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants