Wait! Looks like we don't have enough information to add this to bug database. Please follow this template bug format.
  1. #1
    Sencha Premium Member lorezyra's Avatar
    Join Date
    Dec 2007
    Location
    Japan -- 日本
    Posts
    638
    Vote Rating
    18
    lorezyra will become famous soon enough lorezyra will become famous soon enough

      0  

    Default Application merge error after GitHub pull

    Application merge error after GitHub pull


    REQUIRED INFORMATION

    Architect Build tested:


    version: 3.0.4.1386


    channel: 3.0.1-stable


    platform: 1.3.0.803


    cmd: 4.0.4.84


    framework: Ext JS 4.2.x




    Project Type:
    • ExtJS 4.2
    Description:
    • After pulling from GitHub (via Git-Tower), Application config is lost and changes app name to default "MyApp."
    Steps to reproduce the problem:
    • Pull from Git after teammate pushed change to Architect project while SA is open and project is in development. No conflicts found from Git pull.
    The result that was expected:
    • Updated classes should present no change to the application config.
    The result that occurs instead:
    • Application config is completely lost.
    • Now have two separate Application in Project Inspector.
    • Forced to close SA and restart.
    • Then only a single application category is found.
    • Have to reconfig the properties for appName, appFolder, name, Loader, etc...
    HELPFUL INFORMATION

    Screenshot, Project, or Video:
    • attached
    Screen Shot 2014-06-26 at 2.19.49 PM.jpg

    Possible fix:
    • not provided
    Operating System:
    • MacOS X 10.9.3
    Perfection as a goal is a nice idea that can point one in a specific direction. However, since "perfection" is an ever changing (evolving?) and moving target, one must admit that perfection can never be obtained...

    When in doubt, check the d4mn source code!
    ================================================
    And here are my terms...
    1. I don't care if you use my source code. (Known as "Code.")
    2. I don't care if I get any monetary compensation.
    3. I do care to receive credit for Code provided. So, please keep my name in the comments for Code provided.
    4. Code is provided without warranty "AS-IS" and I claim absolutely no warranty nor liability to the quality, security, and run-ability on any platform.
    5. By using Code, you accept all risk inherit with Code regardless if Code has known and yet to be discovered bugs.
    6. You are welcome to change and improve the Code to best meet your needs.
    7. I don't care if you use the Code in a commercial or open-source project.
    8. You are not required to contact me prior to using the Code.
    ================================================
    Simple. Enjoy.

  2. #2
    Sencha - Architect Dev Team aconran's Avatar
    Join Date
    Mar 2007
    Posts
    9,365
    Vote Rating
    128
    aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold

      0  

    Default


    Can you give us some more information about what in the metadata files was changed? Did they edit something on the application node?

    We'd like to reproduce and resolve it.
    Aaron Conran
    @aconran
    Sencha Architect Development Team

  3. #3
    Sencha Premium Member lorezyra's Avatar
    Join Date
    Dec 2007
    Location
    Japan -- 日本
    Posts
    638
    Vote Rating
    18
    lorezyra will become famous soon enough lorezyra will become famous soon enough

      0  

    Default


    I've been so busy with building new features for my project that I neglected to analyze the commit logs for this...


    Here's the screenshot of Git-Tower 2.0 History at the time of the error:

    スクリーンショット 2014-07-01 10.14.58.jpg


    Then I looked at the next entry, it would appear that my teammate incorrectly resolved the commit conflict...

    スクリーンショット 2014-07-01 10.15.11.jpg



    From what I can see in the commit, two new stores were added and one model's name was changed. Since this would modify both .architect & metadata/Application files, it be different from the teammates local copy. It looks like he choose to merge with the GitHub copy rather than use his local (more up-to-date) copy.

    ** Is there any way for SA to ask the developer if they really want to rebuild/replace the application node due to merge detection?

    I don't normally see this type of behavior, but then my teammate is relatively new to Sencha.
    Perfection as a goal is a nice idea that can point one in a specific direction. However, since "perfection" is an ever changing (evolving?) and moving target, one must admit that perfection can never be obtained...

    When in doubt, check the d4mn source code!
    ================================================
    And here are my terms...
    1. I don't care if you use my source code. (Known as "Code.")
    2. I don't care if I get any monetary compensation.
    3. I do care to receive credit for Code provided. So, please keep my name in the comments for Code provided.
    4. Code is provided without warranty "AS-IS" and I claim absolutely no warranty nor liability to the quality, security, and run-ability on any platform.
    5. By using Code, you accept all risk inherit with Code regardless if Code has known and yet to be discovered bugs.
    6. You are welcome to change and improve the Code to best meet your needs.
    7. I don't care if you use the Code in a commercial or open-source project.
    8. You are not required to contact me prior to using the Code.
    ================================================
    Simple. Enjoy.

  4. #4
    Sencha - Architect Dev Team aconran's Avatar
    Join Date
    Mar 2007
    Posts
    9,365
    Vote Rating
    128
    aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold aconran is a splendid one to behold

      0  

    Default


    Hrm so the error was in the merge and Architect is just picking up what is on disk?

    I think most would find it pretty annoying if Architect was always questioning whether or not what I just did was accurate or not.
    Aaron Conran
    @aconran
    Sencha Architect Development Team

  5. #5
    Sencha Premium Member lorezyra's Avatar
    Join Date
    Dec 2007
    Location
    Japan -- 日本
    Posts
    638
    Vote Rating
    18
    lorezyra will become famous soon enough lorezyra will become famous soon enough

      0  

    Default


    While I would agree, I do prefer to be notified if the changes detected would make numerous changes to the Application node. Especially if a corrupt node is detected. I'm not suggesting that SA confirm on every file change from a repo pull. (Dreamweaver does this and it is annoying.)


    Perhaps a compromise:
    I know that many other non-interpreted languages have well established "best practices" when it comes to CVS/SVN/Git/etc... Perhaps what we should have (if not already documented) is a section on using Git, BitBucket, etc on how to handle merges from the repository.

    Any thoughts?
    Perfection as a goal is a nice idea that can point one in a specific direction. However, since "perfection" is an ever changing (evolving?) and moving target, one must admit that perfection can never be obtained...

    When in doubt, check the d4mn source code!
    ================================================
    And here are my terms...
    1. I don't care if you use my source code. (Known as "Code.")
    2. I don't care if I get any monetary compensation.
    3. I do care to receive credit for Code provided. So, please keep my name in the comments for Code provided.
    4. Code is provided without warranty "AS-IS" and I claim absolutely no warranty nor liability to the quality, security, and run-ability on any platform.
    5. By using Code, you accept all risk inherit with Code regardless if Code has known and yet to be discovered bugs.
    6. You are welcome to change and improve the Code to best meet your needs.
    7. I don't care if you use the Code in a commercial or open-source project.
    8. You are not required to contact me prior to using the Code.
    ================================================
    Simple. Enjoy.

Thread Participants: 1