Skip to content

Added troubleshooting folder#1021

Closed
sincethestudy wants to merge 4 commits intoCCExtractor:masterfrom
sincethestudy:master
Closed

Added troubleshooting folder#1021
sincethestudy wants to merge 4 commits intoCCExtractor:masterfrom
sincethestudy:master

Conversation

@sincethestudy
Copy link
Copy Markdown

@sincethestudy sincethestudy commented Oct 27, 2018

This folder is for users to reference to or add to when an issue is found.

Please prefix your pull request with one of the following: [FEATURE] [FIX] [IMPROVEMENT].

In raising this pull request, I confirm the following (please check boxes):

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.
  • I have mentioned this change in the changelog.

My familiarity with the project is as follows (check one):

  • I have never used CCExtractor.
  • I have used CCExtractor just a couple of times.
  • I absolutely love CCExtractor, but have not contributed previously.
  • I am an active contributor to CCExtractor.

{pull request content here}

@ccextractor-bot
Copy link
Copy Markdown
Collaborator

ccextractor-bot commented Oct 27, 2018

CCExtractor CI platform finished running the test files on linux. Below is a summary of the test results:

Report Name Tests Passed
Broken 13/13
DVB 4/7
DVR-MS 2/2
General 27/27
Hauppage 3/3
MP4 3/3
NoCC 10/10
Teletext 21/21
WTV 13/13
XDS 34/34
CEA-708 14/14
DVD 3/3
Options 86/86

It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you).

Your PR breaks these cases:


Check the result page for more info.

@canihavesomecoffee
Copy link
Copy Markdown
Member

Can you try to make it an .md? That's more universally accessible on GitHub compared to PDF 👍


With only the default build tools installed you might receive an error that looks like this when trying to build:

![alt text](https://github.com/CCExtractor/ccextractor/tree/master/troubleshooting/images/v141_xp-img1)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you write a description of the image instead of saying alt text on all of them?

@ccextractor-bot
Copy link
Copy Markdown
Collaborator

ccextractor-bot commented Oct 27, 2018

CCExtractor CI platform finished running the test files on windows. Below is a summary of the test results:

Report Name Tests Passed
Broken 12/13
DVB 4/7
DVR-MS 2/2
General 27/27
Hauppage 3/3
MP4 3/3
NoCC 10/10
Teletext 14/21
WTV 13/13
XDS 34/34
CEA-708 14/14
DVD 3/3
Options 80/86

It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you).

Your PR breaks these cases:


Check the result page for more info.

Epiphic added 2 commits October 27, 2018 15:23
This folder is for users to reference to or add to when an issue is found.
I added the image resources to display in the .md files
@ghost
Copy link
Copy Markdown

ghost commented Oct 29, 2018

Please see https://github.com/CCExtractor/ccextractor/wiki/Building-CCExtractor-with-Visual-Studio and https://github.com/CCExtractor/ccextractor/wiki/Solving-build-issue-with-the-v141_xp-toolchain which I have done for GCI. I have also embedded the images in these using the proper commonmark formatting.

Copy link
Copy Markdown
Contributor

@anshul1912 anshul1912 left a comment

Choose a reason for hiding this comment

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

only information about solving v141_xp build issue in Visual Studio 2015+ is not present, that i would suggest to put in wiki.
And rest all info is redundant.

Comment thread src/CMakeLists.txt
ELSE(CMAKE_SIZEOF_VOID_P EQUAL 8)
add_definitions(-DVERSION_FILE_PRESENT -DFT2_BUILD_LIBRARY -DGPAC_DISABLE_VTT -DGPAC_DISABLE_OD_DUMP)
ENDIF(CMAKE_SIZEOF_VOID_P EQUAL 8)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Already present in code

add_definitions(-DVERSION_FILE_PRESENT -DFT2_BUILD_LIBRARY -DGPAC_DISABLE_VTT -DGPAC_DISABLE_OD_DUMP)

if(CMAKE_SIZEOF_VOID_P EQUAL 8)
  add_definitions(-DGPAC_64_BITS)
endif()


It should take a few moments, then say build successful:

![alt text](https://github.com/CCExtractor/ccextractor/tree/master/troubleshooting/images/build-img4) No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we already have "https://github.com/CCExtractor/ccextractor/wiki/Installation#windows" I dont think we need this info again


Under optional, select the the two highlighted tools in the picture above. In the bottom right corner it will show the amount of space required for the addition, ensure you have enough space on your disk, then click Modify. If Visual Studio doesn't open up after modifying, open the ccextractor.sln again. Right click the project and click build, and this time it should be successful!

![alt text](https://github.com/CCExtractor/ccextractor/tree/master/troubleshooting/images/v141_xp-img4) No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Instead of new folder, I would suggest to put troubleshooting information to go in wiki.

@cfsmp3 cfsmp3 closed this Mar 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants