0003296: Get rid of install_dependable target.

Chat on our Discord Server Get the latest version: 3.1
Source Code

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0003296

Doomseeker

[All Projects] Bug

public

2017-10-07 10:56

2018-10-27 22:55

Reporter

Zalewa

Assigned To

Zalewa

Priority

normal

Severity

tweak

Reproducibility

sometimes

Status

closed

Resolution

fixed

Platform

Microsoft

Windows

OS Version

XP/Vista/7

Product Version

1.1

Target Version

1.2

Fixed in Version

1.2

Summary

0003296: Get rid of install_dependable target.

Description

install_dependable target causes some grief with multi-process compilation in Visual Studio. I repeatedly get failed compilations that eventually succeed just by recompiling.

install_dependable is "needed" for targets that prepare auto-update packages. It should be figured out how to prepare these packages either without having to INSTALL, or by running INSTALL through different means.

Attached Files

Relationships

Notes
(0019060) Zalewa (developer) 2018-02-16 12:41	It turns out you can just add more than one COMMAND into add_custom_target(), thus chaining commands is possible, thus a separate target for "cmake --build . --target install" is not necessary. 'https://bitbucket.org/Doomseeker/doomseeker/commits/dcc16fecc8423eedba20bc59761b1222bfb5ecd8 [^]' This should be tested on Mac.

(0019074) Blzut3 (administrator) 2018-02-18 21:00	These targets are not used on the Mac since at least at least at the moment the build process for the official Mac binaries requires multiple compilation passes and merging which is captured in makemacdmg.sh. I then plug the resulting app bundle into the ruby scripts.

(0019075) Zalewa (developer) 2018-02-19 00:17	Thanks. I love seeing tickets in the green.

**Issue Community Support**
This issue is already marked as resolved. If you feel that is not the case, please reopen it and explain why.
Supporters:	No one explicitly supports this issue yet.
Opponents:	No one explicitly opposes this issue yet.

Issue History
Date Modified	Username	Field	Change
2017-10-07 10:56	Zalewa	New Issue
2017-10-08 02:48	WubTheCaptain	Category	Suggestion => Bug
2018-02-15 20:37	Zalewa	Assigned To	=> Zalewa
2018-02-15 20:37	Zalewa	Status	new => assigned
2018-02-16 11:40	Zalewa	Summary	Get rid off install_dependable target. => Get rid of install_dependable target.
2018-02-16 12:41	Zalewa	Note Added: 0019060
2018-02-16 12:41	Zalewa	Assigned To	Zalewa => Blzut3
2018-02-16 12:41	Zalewa	Status	assigned => needs testing
2018-02-18 21:00	Blzut3	Note Added: 0019074
2018-02-18 21:01	Blzut3	Status	needs testing => resolved
2018-02-18 21:01	Blzut3	Fixed in Version	=> 1.2
2018-02-18 21:01	Blzut3	Resolution	open => fixed
2018-02-18 21:01	Blzut3	Assigned To	Blzut3 => Zalewa
2018-02-18 21:01	Blzut3	Target Version	=> 1.2
2018-02-19 00:17	Zalewa	Note Added: 0019075
2018-10-27 22:55	WubTheCaptain	Status	resolved => closed

Questions or other issues? Contact Us.

Links

Doom and Doom II are the property of id Software.