Handling errors inside BOOST_PYTHON_MODULE()

I have a module that may fail to initialize due to missing dependencies. So I do

BOOST_PYTHON_MODULE(mymodule)
{
...
    if (missing_dependency) {
        throw std::exception("Missing dependency");
    }
...
}

The exception is correctly caught and PyErr_SetString(PyExc_RuntimeError, x.what()) is called, but python reports

    import mymodule
SystemError: initialization of mymodule raised unreported exception

Why unreported exception? My platform is Windows 10/Python 3.7.4 x64/MSVC 2017 if that matters. python_boost version is 1.69.0.

What happens if you replace std::exception by std::runtime_error or some other more specific exception type ?

No, switching to std::runtime_error does not help, I still have unreported exception. And looks like the exception type has nothing to do with the issue: I set a debug breakpoint in

python/src/errors.cpp

Lines 48 to 51 in ac62db1

    
           catch(const std::exception& x) 
        
           { 
        
               PyErr_SetString(PyExc_RuntimeError, x.what()); 
        
           }

and verified that this catch case was executed.

OK, I see. So the Python error has correctly been set by Boost.Python, but Python's own runtime then flags this as an "unreported exception". In other words, you could strip your code off all Boost.Python references, only using Python's own C API, and you'd still observe the issue.
Can you report this to the Python bug tracker ?

Are you sure that's a Python problem? Maybe

python/src/module.cpp

Line 24 in 1727b67

handle_exception(init_function);

should be replaced with

if (handle_exception(init_function)) return NULL;

?

Oh, indeed, that looks sensible. Can you write a (minimal) test case to demonstrate the problem as well as that your proposed change fixes it ? That would make a great PR !

I just tried to fix it locally this way and it solved the problem. Well, I could try to create a test case and PR, but I'm quite new to boost libs testing infrastructure...

OK, great. I'll see whether I can write a quick test.

@stefanseefeld

Hi @stefanseefeld
Any news on the issue? Are you still planning to create a test and finally commit the fix?

@db4

Had the same issue, boost::python was hiding the real exception with raised unreported exception. I applied the patch if (handle_exception(init_function)) return NULL; mentioned by @db4 and now the real exception is thrown.

Also just ran into this problem.. Just bumping this issue to show it's still relevant and affects users.

Add a test case for issue boostorg#280

Fix issue boostorg#280

Also just ran into this problem.. Just bumping this issue to show it's still relevant and affects users.

I finally decided to stop waiting for something and create PR myself

Add a test case for issue #280

Fix issue #280

Navigation Menu

Handling errors inside BOOST_PYTHON_MODULE() #280

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	catch(const std::exception& x)
	{
	PyErr_SetString(PyExc_RuntimeError, x.what());
	}

Handling errors inside BOOST_PYTHON_MODULE() #280

Description

Activity

stefanseefeld commented on Dec 9, 2019

db4 commented on Dec 9, 2019

stefanseefeld commented on Dec 9, 2019

db4 commented on Dec 9, 2019

stefanseefeld commented on Dec 9, 2019

db4 commented on Dec 9, 2019

stefanseefeld commented on Dec 9, 2019

db4 commented on Jun 29, 2020

tdelame commented on Nov 20, 2020

twvd commented on Feb 2, 2021

db4 commented on Feb 3, 2021

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions