gregmac
Posts: 13
Joined: Wed Oct 13, 2010 2:05 am

Detect background changes to .aip and prompt to reload

This is a missing feature that is potentially very dangerous in a multi-developer environment using source control.

Advanced Installer should monitor the .aip file it has open for changes in the background. If changes are detected, it should prompt to reload the file. There are two different prompts that can be shown: One when there are no unsaved changes, and a more severe prompt when there are unsaved changes (eg: either reload from disk and lose your changes, or ignore the changes from the disk version and keep your changes). VisualStudio does this, for example.

May not be that common for two devs to be working on installers, but I can see it happening in a large-ish product made of many components (like mine).

Scenario:

Two developers using subversion and tortoisesvn.
  • Developer A opens .aip file, and makes some changes
  • Developer B opens .aip file, and makes some changes to a different part, and saves it.
  • Developer B commits to SVN.
  • Developer A is working on several other pieces of product, and as a matter of consequence does an svn update - which gets them the .aip file with Developer B's changes.
  • Developer A finishes working on .aip, and saves it.
At this point, Developer B's changes are now lost (well, once Developer A commits) because Developer A never re-loaded the file, and worst of all, there is no indication that this happened. The only way to tell would be to notice that Developer B's changes never showed up, or by looking at the diffs of the two commits and seeing that commit #2 that A did removed the changes that B did in commit #1 (and somehow know that it wasn't intentional).
Last edited by gregmac on Fri Oct 15, 2010 7:06 pm, edited 1 time in total.
Cosmin
Posts: 5797
Joined: Tue Jul 10, 2007 6:39 pm
Contact: Website

Re: Detect background changes to .aip and prompt to reload

Hi,

I added this improvement on our TODO list and it will be available in a future version. Thank you for your suggestion.

Regards,
Cosmin
Cosmin Pirvu - Advanced Installer Team
Follow us: Twitter - Facebook - YouTube

Return to “Feature Requests”