Skip to content

Added Dockerize PR from main repo - #4

Merged
xPTM1219 merged 4 commits into
masterfrom
dockerize-scalpel
Jan 3, 2026
Merged

Added Dockerize PR from main repo#4
xPTM1219 merged 4 commits into
masterfrom
dockerize-scalpel

Conversation

@xPTM1219

@xPTM1219 xPTM1219 commented Feb 4, 2025

Copy link
Copy Markdown
Owner

Description

Added the Dockerize pull request from the main repo. Also, added comments recommendations.
Additionally,

  • I've tested the building and running the Docker image
  • The C++ warning and error mentioned below will be fixed in pull request 3
  • Updated instructions
  • Ignored recovery folder

TODO

The list was done. Leaving for historic purposes. See update above.

  • Verify image and folder location
  • The recovery folder can be created if it does not exists
  • Fix make error
  • Test

Issues

While trying to build the image it gives the same problem when running make.
See pull request 3 for details.

The error from Docker is the following:

15.37 In file included from scalpel.h:72,
15.37                  from libscalpel_test.cpp:23:
15.37 input_reader.h:82:1: warning: type qualifiers ignored on function return type [-Wignored-quali
fiers]
15.37    82 | const char scalpelInputIsOpen(ScalpelInputReader * const reader);
15.37       | ^~~~~
15.37 In file included from libscalpel_test.cpp:23:
15.37 scalpel.h:306:19: error: ISO C++17 does not allow dynamic exception specifications
15.37   306 |                 ) throw (std::runtime_error);
15.37       |                   ^~~~~
15.46 make[1]: *** [Makefile:517: libscalpel_test.o] Error 1
15.46 make[1]: Leaving directory '/scalpel/src'
15.46 make: *** [Makefile:533: all-recursive] Error 1
------
Dockerfile:22
--------------------
  20 |     COPY . /scalpel
  21 |     WORKDIR /scalpel
  22 | >>> RUN ./bootstrap && ./configure --disable-shared && make
  23 |     ENTRYPOINT ["/scalpel/entrypoint.sh"]
  24 |
--------------------
ERROR: failed to solve: process "/bin/sh -c ./bootstrap && ./configure --disable-shared && make" did
 not complete successfully: exit code: 2

Resource

@xPTM1219
xPTM1219 merged commit b4d09d0 into master Jan 3, 2026
@xPTM1219
xPTM1219 deleted the dockerize-scalpel branch January 3, 2026 20:43
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.

1 participant