Ticket #3267 (closed: wontfix)

Opened 9 years ago

Last modified 5 years ago

Update class_maker.py to work with projects in VATES

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: minor Milestone: Iteration 29
Component: Tools Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

Right now, everything assumes "Framework" as the base folder.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Janik Zikovsky

  • Status changed from accepted to verify
  • Resolution set to fixed

(In [12893]) Fixes #3267: class_maker for Vates folder.

comment:3 Changed 9 years ago by Janik Zikovsky

(In [12894]) Refs #3267]

comment:4 Changed 9 years ago by Janik Zikovsky

To test: It should be possible to make files under Vates/VatesAPI/ using the script.

comment:5 Changed 9 years ago by Stuart Campbell

  • Status changed from verify to verifying
  • Tester set to Stuart Campbell
  • Milestone changed from Iteration 30 to Iteration 29

Moving back to iteration 29 as its been done on the trunk before the release

comment:6 Changed 9 years ago by Stuart Campbell

  • Status changed from verifying to reopened
  • Resolution fixed deleted

It does seem to add a repeated INC/SRC/TEST_FILES section to the CMakeList.txt file for the new class.

I typed (from the trunk/Code/Mantid directory): 

./Build/class_maker.py --vates VatesAPI test

and I ended up with 

set ( INC_FILES
	inc/MantidVatesAPI/test.h
)
set ( SRC_FILES
	src/test.cpp
)
set ( TEST_FILES
	test/testTest.h
)

at the end of the file in addition to these sections at the top.

comment:7 Changed 9 years ago by Janik Zikovsky

  • Status changed from reopened to accepted

comment:8 Changed 9 years ago by Janik Zikovsky

(In [12896]) Refs #3267: Reverted.

comment:9 Changed 9 years ago by Janik Zikovsky

  • Status changed from accepted to verify
  • Resolution set to wontfix

Changed my mind about this. Wontfix.

comment:10 Changed 9 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester changed from Stuart Campbell to Michael Reuter

comment:11 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to closed

OK.

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​4114

Note: See TracTickets for help on using tickets.