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

When changing the feature model, markers are not updated. #700

Closed
skrieter opened this issue Feb 2, 2018 · 9 comments
Closed

When changing the feature model, markers are not updated. #700

skrieter opened this issue Feb 2, 2018 · 9 comments

Comments

@skrieter
Copy link
Collaborator

skrieter commented Feb 2, 2018

Prerequisitives

  • Eclipse Version: Neon.2
  • FeatureIDE-Version: 3.4
  • Operating system: Windows
  • Java runtime used to run eclipse: 1.8
  • FeatureIDE Composer: all
  • Git branch/Commit-ID: develop
  • Java compiler used to build the FeatureIDE plugin:

Issue description

Changing a feature model (e.g., removing a void constraint) does not clear the respective error markers.
Eclipse should perform a clean and (maybe) a build after saving a modified feature model.

@skrieter skrieter added this to the 3.5.0 (March) milestone Feb 2, 2018
@tthuem tthuem modified the milestones: 3.5.0 (March), 3.4.3 (January) Feb 5, 2018
@tthuem
Copy link
Member

tthuem commented Feb 5, 2018

@skrieter Do we need this for the tutorial?

@Kogoro
Copy link
Contributor

Kogoro commented Feb 8, 2018

@tthuem We checked the issue and these were the only markers:
fm5
Even after a build, this does not change.

@Kogoro Kogoro closed this as completed in a42dbbd Feb 8, 2018
@tthuem
Copy link
Member

tthuem commented Feb 9, 2018

The change in the code does not look right. Also, why do those markers not appear anymore?

@PaulWestphal Paul, could you please check whether it works for the tutorial now? (i.e., there are only markers that we need for the tutorial)

@PaulWestphal
Copy link
Contributor

PaulWestphal commented Feb 9, 2018

Hands On II problems:

Occasionally when loading Elevator v1.0, I don't get the "The feature model is void" marker. This doesn't happen very often.

The "Configuration 'x' is invalid" markers don't show up at all when the feature model is void unless I save the feature model without fixing it.

After completing step 4. in the hands on, Info markers sometimes appear up to three times.

tutorial1

e: I just got the info markers seven times by just following the steps of the Hands On, but I'm still unsure how to reproduce this.

tutorial1

@tthuem
Copy link
Member

tthuem commented Feb 9, 2018

Note sure what is going on here, but we are not getting closer to the version for the tutorial.

@Kogoro @subaro Would be good if you could test the complete tutorial, too.

@skrieter We might need you help to get a stable version in time.

@Kogoro
Copy link
Contributor

Kogoro commented Feb 9, 2018

Regarding the not showing void marker: The sat solver delivers true as a result of the isValid() call. Then the marker is not created.
We will take a look at the configuration markers.

@Kogoro
Copy link
Contributor

Kogoro commented Feb 9, 2018

I made a quick fix for the markers. On build, there will be called a method for checking the most common problems.
@PaulWestphal @subaro Please test on the quickfix branch.

@PaulWestphal
Copy link
Contributor

When loading Elevator v1.0 on the quickfix branch "building workspace" never stops. That makes it impossible to run the elevator.

@Kogoro
Copy link
Contributor

Kogoro commented Feb 11, 2018

5366e89 should solve the rebuild issue. It appeared only sometimes when the feature model was overridden.

@tthuem tthuem modified the milestones: 3.4.3 (February), 3.5.0 (March) Feb 12, 2018
@Kogoro Kogoro closed this as completed Feb 14, 2018
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

5 participants