search

mathworks / jenkins-matlab-plugin

This plugin enables you to run MATLAB® and Simulink® as part of your Jenkins™ build.
47 stars 54 forks source link

Confusing Build results when using previous versions(23b) of MATLAB #283

Closed Vahila closed 9 months ago

Vahila commented 9 months ago

When Jenkins job is run in 23b or prior, the MATLAB Build Results shows that 0 tasks are run(even though build tasks are run) which is a bit misleading. The user needs to know that the results are populated correctly with 24a

nbhoski commented 9 months ago

May be this could be documented at some place. The design in itself runs the build visulization irrespective of MATLAB version. Also we need to know what are the plans for backporting Davids changes to MATLAB versions that support buildtool.

nbhoski commented 9 months ago

@davidbuzinski do we have any plans pathich previous MATLAB releases for supporting explicit plugin injection ?

davidbuzinski commented 9 months ago

@davidbuzinski do we have any plans pathich previous MATLAB releases for supporting explicit plugin injection ?

The patches have been submitted for all previous releases including buildtool. I think there should be an update that includes the changes for most releases now. Make sure to update to the latest update release to see the changes

mw-hrastega commented 9 months ago

May be this could be documented at some place. The design in itself runs the build visulization irrespective of MATLAB version. Also we need to know what are the plans for backporting Davids changes to MATLAB versions that support buildtool.

Hi, @nbhoski. Can you first try @davidbuzinski's answer? If the issue doesn't get resolved, then we can find a way to point to this bug/limitation in the doc.

nbhoski commented 9 months ago

@davidbuzinski do we have any plans pathich previous MATLAB releases for supporting explicit plugin injection ?

The patches have been submitted for all previous releases including buildtool. I think there should be an update that includes the changes for most releases now. Make sure to update to the latest update release to see the changes

@Vahila could you try this and let me know if this issue persists

Vahila commented 9 months ago

@davidbuzinski @nbhoski MATLAB Build Results are populated correctly for 23b U6.

For 23a U6, there is an error as below: image I see similar error in 23a GR as well image

For22b U7, the error is as below: image

I have checked till 22b as buildtool is supported from 22b.

davidbuzinski commented 9 months ago

@davidbuzinski @nbhoski MATLAB Build Results are populated correctly for 23b U6.

For 23a U6, there is an error as below: image I see similar error in 23a GR as well image

For22b U7, the error is as below: image

I have checked till 22b as buildtool is supported from 22b.

It looks like the buildfile.m that is being tested is using features from buildtool that weren't released yet at the time of several of the releases being tested

Vahila commented 9 months ago

I can see the Build Results getting populated correctly for 22bU8, 23aU6, 23bU6.

nbhoski commented 9 months ago

@Vahila can we close this in that case

Vahila commented 9 months ago

Can we also discuss this in the team meeting? Based on that we can take an action.

Vahila commented 9 months ago

Closing this issue as we have another issue to discuss about the warning message: https://github.com/mathworks/jenkins-matlab-plugin/issues/288