Skip to content

Building - Remove GLTF files from XDEDRAW#649

Merged
dpasukhi merged 1 commit intoOpen-Cascade-SAS:masterfrom
dpasukhi:gltf-no-json-build
Jul 28, 2025
Merged

Building - Remove GLTF files from XDEDRAW#649
dpasukhi merged 1 commit intoOpen-Cascade-SAS:masterfrom
dpasukhi:gltf-no-json-build

Conversation

@dpasukhi
Copy link
Copy Markdown
Member

No description provided.

@dpasukhi dpasukhi added this to the Release 8.0 milestone Jul 28, 2025
@dpasukhi dpasukhi requested a review from Copilot July 28, 2025 19:18
@dpasukhi dpasukhi self-assigned this Jul 28, 2025
@dpasukhi dpasukhi added the 1. Build Building process of OCCT or samples label Jul 28, 2025
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes GLTF file support from the XDEDRAW module by eliminating the inclusion of GLTF-related configuration headers.

  • Removes the DEGLTF_ConfigurationNode.hxx include from the XDEDRAW source file
Comments suppressed due to low confidence (1)

src/Draw/TKXDEDRAW/XDEDRAW/XDEDRAW.cxx:39

  • Removing this include may cause compilation errors if DEGLTF_ConfigurationNode is used elsewhere in this file. Ensure that all references to DEGLTF_ConfigurationNode and related GLTF functionality have been removed from the implementation.
#include <DEOBJ_ConfigurationNode.hxx>

@dpasukhi dpasukhi changed the base branch from IR to master July 28, 2025 20:48
@dpasukhi dpasukhi merged commit 7d2cce0 into Open-Cascade-SAS:master Jul 28, 2025
23 checks passed
@dpasukhi dpasukhi deleted the gltf-no-json-build branch July 28, 2025 20:48
@github-project-automation github-project-automation bot moved this from Todo to Done in Maintenance Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1. Build Building process of OCCT or samples

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants