diff --git a/src/sage/features/__init__.py b/src/sage/features/__init__.py index a5d5ff3c409..df76ec9da99 100644 --- a/src/sage/features/__init__.py +++ b/src/sage/features/__init__.py @@ -606,7 +606,7 @@ class FileFeature(Feature): To work with the file described by the feature, use the method :meth:`absolute_filename`. A :class:`FeatureNotPresentError` is raised if the file cannot be found:: - sage: Executable(name="does-not-exist", executable="does-not-exist-xxxxyxyyxyy").absolute_path() + sage: Executable(name="does-not-exist", executable="does-not-exist-xxxxyxyyxyy").absolute_filename() Traceback (most recent call last): ... sage.features.FeatureNotPresentError: does-not-exist is not available. @@ -654,32 +654,6 @@ def absolute_filename(self) -> str: # the distribution sagemath-environment. raise NotImplementedError - def absolute_path(self): - r""" - Deprecated alias for :meth:`absolute_filename`. - - Deprecated to make way for a method of this name returning a ``Path``. - - EXAMPLES:: - - sage: from sage.features import Executable - sage: Executable(name="sh", executable="sh").absolute_path() - doctest:warning... - DeprecationWarning: method absolute_path has been replaced by absolute_filename - See https://github.com/sagemath/sage/issues/31292 for details. - '/...bin/sh' - """ - try: - from sage.misc.superseded import deprecation - except ImportError: - # The import can fail because sage.misc.superseded is provided by - # the distribution sagemath-objects, which is not an - # install-requires of the distribution sagemath-environment. - pass - else: - deprecation(31292, 'method absolute_path has been replaced by absolute_filename') - return self.absolute_filename() - class Executable(FileFeature): r""" @@ -764,7 +738,7 @@ def absolute_filename(self) -> str: A :class:`FeatureNotPresentError` is raised if the file cannot be found:: - sage: Executable(name="does-not-exist", executable="does-not-exist-xxxxyxyyxyy").absolute_path() + sage: Executable(name="does-not-exist", executable="does-not-exist-xxxxyxyyxyy").absolute_filename() Traceback (most recent call last): ... sage.features.FeatureNotPresentError: does-not-exist is not available. diff --git a/src/sage/features/join_feature.py b/src/sage/features/join_feature.py index afb41e7f227..fd993e90e1b 100644 --- a/src/sage/features/join_feature.py +++ b/src/sage/features/join_feature.py @@ -98,36 +98,6 @@ def _is_present(self): return test return FeatureTestResult(self, True) - def is_functional(self): - r""" - Test whether the join feature is functional. - - This method is deprecated. Use :meth:`Feature.is_present` instead. - - EXAMPLES:: - - sage: from sage.features.latte import Latte - sage: Latte().is_functional() # optional - latte_int - doctest:warning... - DeprecationWarning: method JoinFeature.is_functional; use is_present instead - See https://github.com/sagemath/sage/issues/33114 for details. - FeatureTestResult('latte_int', True) - """ - try: - from sage.misc.superseded import deprecation - except ImportError: - # The import can fail because sage.misc.superseded is provided by - # the distribution sagemath-objects, which is not an - # install-requires of the distribution sagemath-environment. - pass - else: - deprecation(33114, 'method JoinFeature.is_functional; use is_present instead') - for f in self._features: - test = f.is_functional() - if not test: - return test - return FeatureTestResult(self, True) - def hide(self): r""" Hide this feature and all its joined features. diff --git a/src/sage/plot/animate.py b/src/sage/plot/animate.py index c11c80151e2..2dbc162d07c 100644 --- a/src/sage/plot/animate.py +++ b/src/sage/plot/animate.py @@ -1023,8 +1023,9 @@ def ffmpeg(self, savefile=None, show_path=False, output_format=None, # afterwards. Hence 'early_options' and 'ffmpeg_options' # The `-nostdin` is needed to avoid the command to hang, see # https://stackoverflow.com/questions/16523746/ffmpeg-hangs-when-run-in-background - cmd = 'cd {}; ffmpeg -nostdin -y -f image2 {} -i {} {} {}'.format( - shlex.quote(pngdir), early_options, shlex.quote(pngs), ffmpeg_options, shlex.quote(savefile)) + cmd = 'cd {}; {} -nostdin -y -f image2 {} -i {} {} {}'.format( + shlex.quote(pngdir), shlex.quote(FFmpeg().absolute_filename()), + early_options, shlex.quote(pngs), ffmpeg_options, shlex.quote(savefile)) from subprocess import check_call, CalledProcessError, PIPE try: if sage.misc.verbose.get_verbose() > 0: