Skip to content

Issue #4833 nytprof - #4838

Closed
bschmalhofer wants to merge 2 commits into
rel-10_1from
issue-#4833-nytprof
Closed

Issue #4833 nytprof#4838
bschmalhofer wants to merge 2 commits into
rel-10_1from
issue-#4833-nytprof

Conversation

@bschmalhofer

Copy link
Copy Markdown
Contributor

No description provided.

@bschmalhofer
bschmalhofer requested a review from svenoe November 13, 2025 16:30
@bschmalhofer

Copy link
Copy Markdown
Contributor Author

@svenoe : Can you take a look at this? I tend to generally install Devel::NYTProf in the image. The number of dependencies is lower than I expected. See https://github.com/RotherOSS/otobo/compare/rel-10_1...issue-%234833-nytprof?expand=1#diff-fc00e6ffb3c763013c9e771e6f98e462f03770f325ecfb81ec8408a8d663d47b .

Comment thread cpanfile.docker Outdated

# };

# feature 'devel:profiling', 'Modules for profiline Perl code' => sub {

@bschmalhofer bschmalhofer Jan 26, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

"profiline" should be "profiling"

Note that these modules, e.g. Devel::NYTProf, are not installed per default
in the Docker image
this activates profiling. This works just like activation the shotgun option.
@bschmalhofer
bschmalhofer force-pushed the issue-#4833-nytprof branch from b3207af to acb1a67 Compare April 3, 2026 13:59
@bschmalhofer
bschmalhofer deleted the issue-#4833-nytprof branch April 3, 2026 14:10
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