Red Gate forums :: View topic - Clean-up saproj XML
Return to www.red-gate.com RSS Feed Available

Search  | Usergroups |  Profile |  Messages |  Log in  Register 
Go to product documentation
SmartAssembly 6
SmartAssembly 6 forum

Clean-up saproj XML

Search in SmartAssembly 6 forum
Post new topic   Reply to topic
Jump to:  
Author Message
Wernight



Joined: 28 Jun 2012
Posts: 5

PostPosted: Thu Jun 28, 2012 12:19 pm    Post subject: Clean-up saproj XML Reply with quote

I found SmartAssembly doesn't have a complete documentation of the XML project files. More importantly when changing some settings and reverting that change, the XML changes. For example a:

Code:
<Embedded />


becomes

Code:
<Embedded Compress="1" GAC="0" />


Meaning the default values are added. This is a pretty big issue with source control systems. I'd suggest to always or never include the default values.

Also it seems that there is no way to clean-up assemblies that were referenced and are not more referenced (unless you manually edit the saproj file and check each one by one).
Back to top
View user's profile Send private message
Wernight



Joined: 28 Jun 2012
Posts: 5

PostPosted: Thu Jun 28, 2012 12:58 pm    Post subject: Reply with quote

Ideally the SA project editor should have 3 states for such. One being the default, the rest being the defined value. This is only supposing that you wish to possibly change the default value later on. If the default value stay for backward compatibility then better never include the default value. This would be clear.

I forgot to specify that the hardest part in regards to source control systems, is that when merging or doing changes it's impossible to know if those values mean the same as not specifying any. This is because there is no reference XML documentation for it.
Back to top
View user's profile Send private message
james.billings



Joined: 16 Jun 2010
Posts: 1117
Location: My desk.

PostPosted: Fri Jun 29, 2012 1:17 pm    Post subject: Reply with quote

Thanks for your post- I'm guessing for the "affecting source control systems" problem this is perhaps something that wasn't considered for the initial design. Are you finding multiple users are changing the file and it's hard to re-merge or something? I'd like to understand the exact difficulty it's causing.

As for cleaning up assemblies, the GUI has a "rescan dependencies" option within the merging and embedding section that should correct any missing ones, you can then re-save the project.
Back to top
View user's profile Send private message
Wernight



Joined: 28 Jun 2012
Posts: 5

PostPosted: Mon Jul 02, 2012 9:41 am    Post subject: Reply with quote

Rescan adds missing ones but doesn't to remove those that are not used anymore and that don't exist anymore.

Yes it affects greatly merges but also change tracking/code review. When you had something using default attributes values and then it gets changed to something with attributes explicitly defined, you don't know if those changes change anything and what they change.

Exanpe:

- <Foo />
+ <Foo Bar="1" />

Does it mean the "Bar" is now enabled and wasn't before? What affect does "Bar" mean anyway?
Back to top
View user's profile Send private message
james.billings



Joined: 16 Jun 2010
Posts: 1117
Location: My desk.

PostPosted: Thu Jul 05, 2012 1:52 pm    Post subject: Reply with quote

Thanks for the clarification.
I can certainly add a request that we amend the file to always explicitly define the attributes rather than defaulting to "something" when they are missing. I'm not sure when such a change would take place but it'll be on the radar at least.

Thanks!
Back to top
View user's profile Send private message
Display posts from previous:   
Reply to topic All times are GMT + 1 Hour
Page 1 of 1

 
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum


Powered by phpBB © 2001, 2005 phpBB Group