September 2007 - YaST Commits - openSUSE Mailing Lists

[yast-commit] r41147 - in /trunk/storage: package/yast2-storage.changes storage/src/include/custom_part_lib.ycp
by fehr＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: fehr Date: Thu Sep 27 16:22:26 2007 New Revision: 41147 URL: http://svn.opensuse.org/viewcvs/yast?rev=41147&view=rev Log: suppress install media when reading fstab (#327571) Modified: trunk/storage/package/yast2-storage.changes trunk/storage/storage/src/include/custom_part_lib.ycp Modified: trunk/storage/package/yast2-storage.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/storage/package/yast2-storage.ch… ============================================================================== --- trunk/storage/package/yast2-storage.changes (original) +++ trunk/storage/package/yast2-storage.changes Thu Sep 27 16:22:26 2007 @@ -1,7 +1,8 @@ ------------------------------------------------------------------- -Thu Sep 27 11:18:15 CEST 2007 - fehr(a)suse.de +Thu Sep 27 16:21:28 CEST 2007 - fehr(a)suse.de - fix wrong display of cryptotab instead of fstab +- suppress install media when reading fstab (#327571) ------------------------------------------------------------------- Wed Sep 26 12:00:06 CEST 2007 - fehr(a)suse.de Modified: trunk/storage/storage/src/include/custom_part_lib.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/storage/storage/src/include/cust… ============================================================================== --- trunk/storage/storage/src/include/custom_part_lib.ycp (original) +++ trunk/storage/storage/src/include/custom_part_lib.ycp Thu Sep 27 16:22:26 2007 @@ -142,6 +142,10 @@ }); y2milestone( "findExistingFstab tmp %1", tmp ); } + integer s = size(tmp); + tmp = filter( map p, tmp, ``(Storage::CanEdit(p,false))); + if( s!=size(tmp) ) + y2milestone( "findExistingFstab tmp %1", tmp ); if( size(vardev)>0 ) Storage::Umount( vardev ); } -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41146 - in /trunk/product-creator: package/yast2-product-creator.changes src/product-creator.ycp
by lslezak＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: lslezak Date: Thu Sep 27 16:06:28 2007 New Revision: 41146 URL: http://svn.opensuse.org/viewcvs/yast?rev=41146&view=rev Log: - fixed a typo in the description of "passphrase_file" option (#309537) Modified: trunk/product-creator/package/yast2-product-creator.changes trunk/product-creator/src/product-creator.ycp Modified: trunk/product-creator/package/yast2-product-creator.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/product-creator/package/yast2-pr… ============================================================================== --- trunk/product-creator/package/yast2-product-creator.changes (original) +++ trunk/product-creator/package/yast2-product-creator.changes Thu Sep 27 16:06:28 2007 @@ -1,4 +1,10 @@ ------------------------------------------------------------------- +Thu Sep 27 16:03:04 CEST 2007 - lslezak(a)suse.cz + +- fixed a typo in the description of "passphrase_file" option + (#309537) + +------------------------------------------------------------------- Mon Sep 24 08:44:56 CEST 2007 - jsuchome(a)suse.cz - do not close module after kiwi failure (#327593) Modified: trunk/product-creator/src/product-creator.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/product-creator/src/product-crea… ============================================================================== --- trunk/product-creator/src/product-creator.ycp (original) +++ trunk/product-creator/src/product-creator.ycp Thu Sep 27 16:06:28 2007 @@ -58,7 +58,7 @@ ], "passphrase_file" : $[ // command line help text for the 'passhrase_file' option - "help" : _("File with GPG passhrase required for signing the source"), + "help" : _("File with GPG passphrase required for signing the source"), "type" : "string" ], "configfile" : $[ -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41145 - in /branches/tmp/sh/mod-ui/ncurses/src: NCInputField.cc NCInputField.h README_mod-ui
by gs＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: gs Date: Thu Sep 27 15:55:55 2007 New Revision: 41145 URL: http://svn.opensuse.org/viewcvs/yast?rev=41145&view=rev Log: remove 'mayedit' - is not supported (was never supported) Modified: branches/tmp/sh/mod-ui/ncurses/src/NCInputField.cc branches/tmp/sh/mod-ui/ncurses/src/NCInputField.h branches/tmp/sh/mod-ui/ncurses/src/README_mod-ui Modified: branches/tmp/sh/mod-ui/ncurses/src/NCInputField.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCI… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCInputField.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCInputField.cc Thu Sep 27 15:55:55 2007 @@ -44,7 +44,6 @@ unsigned maxFld ) : YInputField( parent, nlabel, passwordMode ) , NCWidget( parent ) - , mayedit( true ) , passwd( passwordMode ) , lwin( 0 ) , twin( 0 ) @@ -62,16 +61,8 @@ ( !maxFldLength || maxFldLength > maxInputLength ) ) { maxFldLength = maxInputLength; } - // FIXME -#if 0 - if ( opt.isEditable.defined() ) - mayedit = opt.isEditable.value(); -#endif setLabel( nlabel ); hotlabel = &label; - // not longer on params - //setText( ntext ); - } /////////////////////////////////////////////////////////////////// @@ -423,15 +414,10 @@ // NCursesEvent NCInputField::wHandleInput( wint_t key ) { - NCursesEvent ret; + NCursesEvent ret = NCursesEvent::none; bool beep = false; bool update = true; - if ( !mayedit ) - { - return NCursesEvent::none; - } - switch ( key ) { case KEY_BACKSPACE: Modified: branches/tmp/sh/mod-ui/ncurses/src/NCInputField.h URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCI… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCInputField.h (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCInputField.h Thu Sep 27 15:55:55 2007 @@ -48,7 +48,6 @@ private: - bool mayedit; bool passwd; NClabel label; wstring buffer; Modified: branches/tmp/sh/mod-ui/ncurses/src/README_mod-ui URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/REA… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/README_mod-ui (original) +++ branches/tmp/sh/mod-ui/ncurses/src/README_mod-ui Thu Sep 27 15:55:55 2007 @@ -85,7 +85,7 @@ NCDialog -------- -TODO: Check whether NCPopupInfo is created/deleted correctly (YDialog::deleteTopmostDialog() missing or not?) +don't call 'delete' for helpPopup use deleteTopmostDialog() instead NCPopupInfo ----------- -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41144 - /branches/tmp/sh/mod-ui/ncurses/src/NCPopupInfo.cc
by gs＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: gs Date: Thu Sep 27 15:39:47 2007 New Revision: 41144 URL: http://svn.opensuse.org/viewcvs/yast?rev=41144&view=rev Log: correct string value Modified: branches/tmp/sh/mod-ui/ncurses/src/NCPopupInfo.cc Modified: branches/tmp/sh/mod-ui/ncurses/src/NCPopupInfo.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCP… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCPopupInfo.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCPopupInfo.cc Thu Sep 27 15:39:47 2007 @@ -54,7 +54,7 @@ , vDim( 20 ) , visible ( false ) { - createLayout( headline->toString(), text->toString(), okButtonLabel, cancelButtonLabel ); + createLayout( headline->value(), text->value(), okButtonLabel, cancelButtonLabel ); } /////////////////////////////////////////////////////////////////// -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41143 - /branches/tmp/sh/mod-ui/ncurses/src/NCDialog.cc
by gs＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: gs Date: Thu Sep 27 15:38:48 2007 New Revision: 41143 URL: http://svn.opensuse.org/viewcvs/yast?rev=41143&view=rev Log: don't call 'delete' for helpPopup use deleteTopmostDialog() instead Modified: branches/tmp/sh/mod-ui/ncurses/src/NCDialog.cc Modified: branches/tmp/sh/mod-ui/ncurses/src/NCDialog.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCD… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCDialog.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCDialog.cc Thu Sep 27 15:38:48 2007 @@ -187,31 +187,6 @@ delete helpPopup; } } -#if 0 -/////////////////////////////////////////////////////////////////// -// -// -// METHOD NAME : NCDialog::nicesize -// METHOD TYPE : long -// -// DESCRIPTION : -// -long NCDialog::nicesize( YUIDimension dim ) -{ - if ( hasDefaultSize() || !childrenCount() ) { - return dim == YD_HORIZ ? wGetDefsze().W : wGetDefsze().H; - } - - //wsze csze( child( 0 )->nicesize( YD_VERT ), - // child( 0 )->nicesize( YD_HORIZ ) ); - wsze csze( firstChild()->preferredHeight(), - firstChild()->preferredWidth() ); - csze = wsze::min( wGetDefsze(), - wsze::max( csze, wsze( 1 ) ) ); - - return dim == YD_HORIZ ? csze.W : csze.H; -} -#endif int NCDialog::preferredWidth() { @@ -1254,7 +1229,8 @@ else { helpPopup->popdown(); - delete helpPopup; + // don't call 'delete' for helpPopup use deleteTopmostDialog() instead + YDialog::deleteTopmostDialog(); helpPopup = 0; pendingEvent = getHotkeyEvent( ch ); } -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41142 - in /trunk/python-bindings/src: Makefile.am YCP.cc YCPTypes.cc YCPTypes.h YCPTypes/Path.cc YCPTypes/Term.cc YCPTypes/YCPTypesInternal.cc YCPTypes/YCPTypesInternal.h YPython.cc YPython.h
by dfiser＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: dfiser Date: Thu Sep 27 15:34:02 2007 New Revision: 41142 URL: http://svn.opensuse.org/viewcvs/yast?rev=41142&view=rev Log: - Added functions for handling with YCPTypes. - Changed Makefile.am to cover YCPTypes and dependencies implied from that. - YCPTypes integrated into YCP (deleted old implementation of YCP types). - YPython was rewritten to use new YCP types and to simplify interface. Now there is available only two methods for transformation of types between YCP and Python, other methods for this are private. Modified: trunk/python-bindings/src/Makefile.am trunk/python-bindings/src/YCP.cc trunk/python-bindings/src/YCPTypes.cc trunk/python-bindings/src/YCPTypes.h trunk/python-bindings/src/YCPTypes/Path.cc trunk/python-bindings/src/YCPTypes/Term.cc trunk/python-bindings/src/YCPTypes/YCPTypesInternal.cc trunk/python-bindings/src/YCPTypes/YCPTypesInternal.h trunk/python-bindings/src/YPython.cc trunk/python-bindings/src/YPython.h Modified: trunk/python-bindings/src/Makefile.am URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/Makefile.am?… ============================================================================== --- trunk/python-bindings/src/Makefile.am (original) +++ trunk/python-bindings/src/Makefile.am Thu Sep 27 15:34:02 2007 @@ -13,9 +13,15 @@ # plugin, libtool forces 'lib' prefix plugin_LTLIBRARIES = libpy2lang_python.la -noinst_LTLIBRARIES = liby2lang_python.la +noinst_LTLIBRARIES = libYCPTypes.la liby2lang_python.la pythonso_LTLIBRARIES = libYCP.la +libYCPTypes_la_SOURCES = YCPTypes.cc YCPTypes.h \ + YCPTypes/YCPTypesInternal.cc YCPTypes/YCPTypesInternal.h \ + YCPTypes/Path.cc \ + YCPTypes/Symbol.cc \ + YCPTypes/Term.cc + # binary part of the python module libYCP_la_SOURCES = \ $(liby2lang_python_la_SOURCES) \ @@ -43,6 +49,8 @@ -ly2util \ -version-info 2:0 +libYCP_la_LIBADD = libYCPTypes.la + libpy2lang_python_la_LDFLAGS = -version-info 2:0 @@ -50,10 +58,12 @@ # the yast libraries are apparently necessary when we're loaded by python. -libpy2lang_python_la_LIBADD = $(PYTHON_LDFLAGS) \ +libpy2lang_python_la_LDFLAGS = $(PYTHON_LDFLAGS) \ -L$(libdir) -L$(plugindir) \ -lycp -ly2 -ly2util +libpy2lang_python_la_LIBADD = libYCPTypes.la + liby2lang_python_la_LIBADD = $(PYTHON_LDFLAGS) Modified: trunk/python-bindings/src/YCP.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YCP.cc?rev=4… ============================================================================== --- trunk/python-bindings/src/YCP.cc (original) +++ trunk/python-bindings/src/YCP.cc Thu Sep 27 15:34:02 2007 @@ -24,6 +24,8 @@ #include "YPython.h" #include "PythonLogger.h" +#include "YCPTypes.h" + /** * Store pointer to ycp module itself. */ @@ -148,34 +150,6 @@ PyObject *traceback; - char class_type_code[] = - "class YCPType:\n\ - def __init__(self, value=''):\n\ - self.type = \"any\"\n\ - self.value = value\n\ - def identity(self):\n\ - return self.type"; - - char class_symbol_type_code[] = - "class Symbol(YCPType):\n\ - def __init__(self, value):\n\ - self.type = \"symbol\"\n\ - self.value = value"; - - char class_path_type_code[] = - "class Path(YCPType):\n\ - def __init__(self, value):\n\ - self.type = \"path\"\n\ - self.value = value"; - - - char class_term_type_code[] = - "class Term(YCPType):\n\ - def __init__(self, name, *args):\n\ - self.type = \"term\"\n\ - self.name = name\n\ - self.value = args"; - char func_y2internal[] = "def y2internal(message):\n\ file, line, func, txt = traceback.extract_stack(None, 2)[0]\n\ @@ -210,6 +184,8 @@ PyRun_SimpleString("import sys, traceback"); Self = Py_InitModule("ycp", YCPMethods); + initYCPTypes(Self); + traceback = PyImport_AddModule("traceback"); PyModule_AddObject(Self,"traceback",traceback); @@ -218,21 +194,6 @@ PyObject *dict = PyModule_GetDict(Self); PyObject *code; - - - - code = PyRun_String(class_type_code, Py_single_input, dict, dict); - Py_XDECREF(code); - - - code = PyRun_String(class_symbol_type_code, Py_single_input, dict, dict); - Py_XDECREF(code); - - code = PyRun_String(class_path_type_code, Py_single_input, dict, dict); - Py_XDECREF(code); - - code = PyRun_String(class_term_type_code, Py_single_input, dict, dict); - Py_XDECREF(code); code = PyRun_String(func_y2internal, Py_single_input, dict, dict); Py_XDECREF(code); @@ -464,11 +425,8 @@ for (int i=0; i< number_args; i++) { pPythonValue = PyTuple_GetItem(args, i); if (pPythonValue) { - if (!(ypython->PythonTypeToYCPSimpleType(pPythonValue,ycpArg))) { - ycpArg = ypython->fromPythonListToYCPList (pPythonValue); - if (ycpArg.isNull()) - ycpArg = ypython->fromPythonDictToYCPMap (pPythonValue); - } + ycpArg = ypython->PythonTypeToYCPType(pPythonValue); + //convert the first argument to path if (i==0) { if (ycpArg->isString()) { @@ -558,11 +516,7 @@ free(temp); - pReturnValue = ypython->YCPTypeToPythonSimpleType(ycpRetValue); - if (!pReturnValue) - pReturnValue = ypython->fromYCPListToPythonList(ycpRetValue); - if (!pReturnValue) - pReturnValue = ypython->fromYCPMapToPythonDict(ycpRetValue); + pReturnValue = ypython->YCPTypeToPythonType(ycpRetValue); if (pReturnValue) return pReturnValue; @@ -665,15 +619,8 @@ for (int i=2; i< number_args; i++) { pPythonValue = PyTuple_GetItem(args, i); if (pPythonValue) { - if (!(ypython->PythonTypeToYCPSimpleType(pPythonValue,ycpArg))) { - ycpArg = ypython->fromPythonListToYCPList(pPythonValue); - if (ycpArg.isNull()) - ycpArg = ypython->fromPythonDictToYCPMap(pPythonValue); - if (ycpArg.isNull()) - ycpArg = ypython->fromPythonTupleToYCPList(pPythonValue); - if (ycpArg.isNull()) - ycpArg = ypython->fromPythonTermToYCPTerm(pPythonValue); - } + ycpArg = ypython->PythonTypeToYCPType(pPythonValue); + if (fun_type->parameterType(i-2)->matchvalue(ycpArg) != 0) { y2error ("Wrong type of argumment %d",i-2); return PyExc_TypeError; @@ -709,16 +656,7 @@ delete []ns_name; delete []func_name; - pReturnValue = ypython->YCPTypeToPythonSimpleType(ycpRetValue); - if (!pReturnValue) - pReturnValue = ypython->fromYCPListToPythonList(ycpRetValue); - if (!pReturnValue) - pReturnValue = ypython->fromYCPListToPythonTuple(ycpRetValue); - if (!pReturnValue) - pReturnValue = ypython->fromYCPMapToPythonDict(ycpRetValue); - if (!pReturnValue) - pReturnValue = ypython->fromYCPTermToPythonTerm(ycpRetValue); - + pReturnValue = ypython->YCPTypeToPythonType(ycpRetValue); return pReturnValue; Modified: trunk/python-bindings/src/YCPTypes.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YCPTypes.cc?… ============================================================================== --- trunk/python-bindings/src/YCPTypes.cc (original) +++ trunk/python-bindings/src/YCPTypes.cc Thu Sep 27 15:34:02 2007 @@ -1,4 +1,5 @@ #include "YCPTypes.h" +#include "YCPTypes/YCPTypesInternal.h" using std::string; bool initYCPTypes(PyObject *module) @@ -27,7 +28,7 @@ return false; } -YCPType getType(PyObject *obj) +YCPType getYCPType(PyObject *obj) { if (isSymbol(obj)) return SYMBOL; @@ -42,8 +43,7 @@ /***** Symbol *****/ bool isSymbol(PyObject *obj) { - if (PyInstance_Check(obj) - && PyObject_IsInstance(obj, (PyObject *)&SymbolType)) + if (PyObject_IsInstance(obj, (PyObject *)&SymbolType)) return true; return false; } @@ -55,14 +55,27 @@ } return string(); } + +PyObject *Symbol_New(PyObject *value) +{ + return YCPTypeString_New(value, &SymbolType); +} + +PyObject *Symbol_NewString(const char *value) +{ + PyObject *val = Py_BuildValue("s", value); + PyObject *ret = Symbol_New(val); + Py_DECREF(val); + + return ret; +} /***** Symbol END *****/ /***** Path *****/ bool isPath(PyObject *obj) { - if (PyInstance_Check(obj) - && PyObject_IsInstance(obj, (PyObject *)&PathType)) + if (PyObject_IsInstance(obj, (PyObject *)&PathType)) return true; return false; } @@ -74,14 +87,27 @@ } return string(); } + +PyObject *Path_New(PyObject *value) +{ + return YCPTypeString_New(value, &PathType); +} + +PyObject *Path_NewString(const char *value) +{ + PyObject *val = Py_BuildValue("s", value); + PyObject *ret = Path_New(val); + Py_DECREF(val); + + return ret; +} /***** Path END *****/ /***** Term *****/ bool isTerm(PyObject *obj) { - if (PyInstance_Check(obj) - && PyObject_IsInstance(obj, (PyObject *)&TermType)) + if (PyObject_IsInstance(obj, (PyObject *)&TermType)) return true; return false; } @@ -101,5 +127,62 @@ } return Py_None; } + + +PyObject *Term_New(PyObject *name, PyObject *value) +{ + PyObject *ret; + PyObject *args; + int size, i; + PyObject *tmp; + + if (!PyTuple_Check(value) || !PyString_Check(name)){ + return Py_None; + } + + // create args variable + size = PyTuple_Size(value); + args = PyTuple_New(size + 1); + Py_INCREF(name); + if (PyTuple_SetItem(args, 0, name) != 0){ + Py_XDECREF(args); + Py_DECREF(name); + return Py_None; + } + for (i=1; i < size + 1; i++){ + tmp = PyTuple_GetItem(value, i-1); + Py_INCREF(tmp); + if (PyTuple_SetItem(args, i, tmp) != 0){ + Py_XDECREF(args); + Py_DECREF(tmp); + return Py_None; + } + } + + // create new object + ret = Term_new(&TermType, Py_None, Py_None); + if (ret == NULL){ + Py_XDECREF(args); + return Py_None; + } + + // initialize object + if (Term_init((Term *)ret, args, Py_None) == -1){ + Py_XDECREF(args); + return Py_None; + } + + Py_XDECREF(args); + return ret; +} + +PyObject *Term_NewString(const char *name, PyObject *value) +{ + PyObject *nam = Py_BuildValue("s", name); + PyObject *ret = Term_New(nam, value); + Py_DECREF(nam); + + return ret; +} /***** Term END *****/ Modified: trunk/python-bindings/src/YCPTypes.h URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YCPTypes.h?r… ============================================================================== --- trunk/python-bindings/src/YCPTypes.h (original) +++ trunk/python-bindings/src/YCPTypes.h Thu Sep 27 15:34:02 2007 @@ -24,6 +24,13 @@ bool isSymbol(PyObject *); std::string Symbol_getValue(Symbol *); +/** + * Return new object created from str. + * Return New Reference! + */ +PyObject *Symbol_New(PyObject *str); +PyObject *Symbol_NewString(const char *); + /** * Path @@ -34,6 +41,9 @@ bool isPath(PyObject *); std::string Path_getValue(Path *); +PyObject *Path_New(PyObject *str); +PyObject *Path_NewString(const char *); + /** * Term @@ -56,6 +66,9 @@ */ PyObject *Term_getValue(Term *); +PyObject *Term_New(PyObject *name, PyObject *list_value); +PyObject *Term_NewString(const char *name, PyObject *list_value); + /** * Initialize alll YCP types @@ -75,6 +88,6 @@ * } */ enum YCPType { NOT_YCP_TYPE, PATH, SYMBOL, TERM }; -YCPType getType(PyObject *); +YCPType getYCPType(PyObject *); #endif Modified: trunk/python-bindings/src/YCPTypes/Path.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YCPTypes/Pat… ============================================================================== --- trunk/python-bindings/src/YCPTypes/Path.cc (original) +++ trunk/python-bindings/src/YCPTypes/Path.cc Thu Sep 27 15:34:02 2007 @@ -2,37 +2,6 @@ using std::string; /** - * Returns new Path object created from string str. - * Returns New Reference! - */ -static PyObject *Path_StringNew(const char *str) -{ - PyObject *ret; - PyObject *args; - - // create args variable - args = Py_BuildValue("(s)", str); - - // create new Path object - ret = YCPTypeString_new(&PathType, Py_None, Py_None); - if (ret == NULL){ - Py_XDECREF(args); - return Py_None; - } - - // initialize Path object - if (YCPTypeString_init((YCPTypeString *)ret, args, Py_None) == -1){ - Py_XDECREF(args); - return Py_None; - } - - Py_XDECREF(args); - return ret; -} - - - -/** * Compare two Path objects. * If obj1 or obj2 are not Paths returns -1 (it means not equal) */ @@ -75,7 +44,7 @@ new_value += "."; new_value += str; - return Path_StringNew(new_value.c_str()); + return Path_NewString(new_value.c_str()); } /** @@ -96,7 +65,7 @@ new_value += "."; new_value += PyString_AsString(self->value); - return Path_StringNew(new_value.c_str()); + return Path_NewString(new_value.c_str()); } static PyMethodDef Path_methods[] = { Modified: trunk/python-bindings/src/YCPTypes/Term.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YCPTypes/Ter… ============================================================================== --- trunk/python-bindings/src/YCPTypes/Term.cc (original) +++ trunk/python-bindings/src/YCPTypes/Term.cc Thu Sep 27 15:34:02 2007 @@ -14,7 +14,7 @@ * This function is called before Term_init. * In this function is set only default values. */ -static PyObject *Term_new(PyTypeObject *type, PyObject *args, +PyObject *Term_new(PyTypeObject *type, PyObject *args, PyObject *kwds) { Term *self; @@ -83,7 +83,7 @@ /** * Returns hash of Term object. */ -long Term_hash(Term *self) +static long Term_hash(Term *self) { if (self->hash != -1){ self->hash = PyObject_Hash(self->name) + PyObject_Hash(self->value); Modified: trunk/python-bindings/src/YCPTypes/YCPTypesInternal.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YCPTypes/YCP… ============================================================================== --- trunk/python-bindings/src/YCPTypes/YCPTypesInternal.cc (original) +++ trunk/python-bindings/src/YCPTypes/YCPTypesInternal.cc Thu Sep 27 15:34:02 2007 @@ -50,3 +50,38 @@ } return self->hash; } + +PyObject *YCPTypeString_New(PyObject *value, PyTypeObject *type) +{ + PyObject *ret; + PyObject *args; + + if (!PyString_Check(value)){ + return Py_None; + } + + // create args variable + args = PyTuple_New(1); + Py_INCREF(value); + if (PyTuple_SetItem(args, 0, value) != 0){ + Py_XDECREF(args); + Py_DECREF(value); + } + + // create new Path object + ret = YCPTypeString_new(type, Py_None, Py_None); + if (ret == NULL){ + Py_XDECREF(args); + return Py_None; + } + + // initialize Path object + if (YCPTypeString_init((YCPTypeString *)ret, args, Py_None) == -1){ + Py_XDECREF(args); + return Py_None; + } + + Py_XDECREF(args); + return ret; +} + Modified: trunk/python-bindings/src/YCPTypes/YCPTypesInternal.h URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YCPTypes/YCP… ============================================================================== --- trunk/python-bindings/src/YCPTypes/YCPTypesInternal.h (original) +++ trunk/python-bindings/src/YCPTypes/YCPTypesInternal.h Thu Sep 27 15:34:02 2007 @@ -29,4 +29,13 @@ */ long YCPTypeString_hash(YCPTypeString *self); +/** + * Return new object that has type type. + */ +PyObject *YCPTypeString_New(PyObject *str, PyTypeObject *type); + + + +PyObject *Term_new(PyTypeObject *type, PyObject *args, PyObject *kwds); +int Term_init(Term *self, PyObject *args, PyObject *kwds); #endif Modified: trunk/python-bindings/src/YPython.cc URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YPython.cc?r… ============================================================================== --- trunk/python-bindings/src/YPython.cc (original) +++ trunk/python-bindings/src/YPython.cc Thu Sep 27 15:34:02 2007 @@ -1,7 +1,7 @@ /*---------------------------------------------------------------------\ -| | -| __ __ ____ _____ ____ | -| \ \ / /_ _/ ___|_ _|___ \ | +| | +| __ __ ____ _____ ____ | +| \ \ / /_ _/ ___|_ _|___ \ | | \ V / _` \___ \ | | __) | | | | | (_| |___) || | / __/ | | |_|\__,_|____/ |_| |_____| | @@ -47,29 +47,19 @@ #include <ycp/YCPVoid.h> #include <ycp/YCPCode.h> #include <ycp/YCPExternal.h> -YPython * YPython::_yPython = 0; - -YPython::YPython() { - PyObject* pMain; - PyObject* pMainDict; - pMain = PyImport_AddModule("__main__"); - PyRun_SimpleString("from ycp import *"); - pMainDict = PyModule_GetDict(pMain); - pPathClass = PyDict_GetItemString(pMainDict, "Path"); - pSymbolClass = PyDict_GetItemString(pMainDict, "Symbol"); - pTermClass = PyDict_GetItemString(pMainDict, "Term"); - -} +#include "YCPTypes.h" +#include <iostream> +#define DBG(str) \ + std::cerr << __FILE__ << ": " << __LINE__ << ": " << str << std::endl; \ + std::cerr.flush() +YPython * YPython::_yPython = 0; -YPython::~YPython() { +YPython::YPython(){} - Py_CLEAR(pPathClass); - Py_CLEAR(pSymbolClass); - Py_CLEAR(pTermClass); -} +YPython::~YPython(){} YPython * @@ -113,7 +103,7 @@ * evaluate of function from python * @param string name of module * @param string name of function - * @param bool is it method? + * @param bool is it method? TODO !! * @param YCPList argumnets from YCP to python function(0 - dummy) * @return YCPValue return value from python's function */ @@ -121,441 +111,311 @@ YPython::callInner (string module, string function, bool method, YCPList argList) { - + PyObject* pMain; + PyObject* pMainDict; + PyObject* pFunc; + PyObject* pArgs; + PyObject* pReturn; + YCPValue result = YCPNull (); - PyObject* pMain; - PyObject* pMainDict; - PyObject* pFunc; - PyObject* pArgs; - PyObject* pReturn; - - YCPValue result = YCPNull (); - - - pMain = PyImport_AddModule("__main__"); - //PyRun_SimpleString("import __main__"); - pMainDict = PyModule_GetDict(pMain); - - - - pFunc = PyDict_GetItemString(pMainDict, function.c_str()); - - - pArgs = PyTuple_New(argList->size()-1); - - PyObject *pArg; - //Parsing argumments - - y2milestone ("name of function %s and nuber of arguments %d", function.c_str(), argList->size()-1); - - for ( int i=1; i < argList->size(); i++ ) { - //FIXME add checking for all types from YCP and conversion to python types... - - pArg = YCPTypeToPythonSimpleType(argList->value(i)); - - if (!pArg) - pArg = fromYCPListToPythonList(argList->value(i)); - - if (!pArg) - pArg = fromYCPListToPythonTuple(argList->value(i)); - - if (!pArg) - pArg = fromYCPMapToPythonDict(argList->value(i)); - if (!pArg) - pArg = fromYCPTermToPythonTerm(argList->value(i)); - - PyTuple_SetItem(pArgs,i-1, pArg); - - } - - - pReturn = PyObject_CallObject(pFunc, pArgs); - - Py_CLEAR(pArgs); - - - - if (pReturn) { - //result = YCPInteger(PyInt_AsLong(pReturn)); //new YCPValue(); - - if (!PythonTypeToYCPSimpleType(pReturn, result)) - result = fromPythonTupleToYCPList(pReturn); - - if (result.isNull()) - result = fromPythonListToYCPList (pReturn); - - if (result.isNull()) - result = fromPythonDictToYCPMap (pReturn); - - if (result.isNull()) - result = fromPythonTermToYCPTerm(pReturn); - - - } - Py_CLEAR(pReturn); - - if (result.isNull ()) { - y2error ("Result is NULL when returning from %s", function.c_str()); - result = YCPVoid (); - } - - return result; -} - - - /** - * Transfer from python simple type to ycp simple type - * transfered are: boolean, integer, string, float, - **/ -bool YPython::PythonTypeToYCPSimpleType(PyObject* pPythonValue, YCPValue &out) { - PyObject * value; - // boolean value handling - if (PyBool_Check(pPythonValue)) { - PyObject * true_value = PyBool_FromLong(1); - int compare = PyObject_Compare(pPythonValue, true_value); - if (compare == 0) { - out = YCPBoolean (true); - } else if (compare < 1) { - out = YCPBoolean (false); - } else { - return false; - } - return true; - } - - //integer value handling - if (PyInt_Check(pPythonValue)) { - - out = YCPInteger(PyInt_AsLong(pPythonValue)); - return true; - } - - //float value handling - if (PyFloat_Check(pPythonValue)) { - - out = YCPFloat(PyFloat_AsDouble(pPythonValue)); - return true; - } - - //string value handling - if (PyString_Check(pPythonValue)) { - - out = YCPString(PyString_AsString(pPythonValue)); - return true; - } - - if (pPythonValue == Py_None) { - out = YCPNull(); - return true; - } - - if (PyInstance_Check(pPythonValue)) { - if (PyObject_HasAttrString(pPythonValue, "type")) { - PyObject * py_attr = PyObject_GetAttrString(pPythonValue,"type"); - if (PyString_Check(py_attr)) { - string type = PyString_AsString(py_attr); - if (type.compare("symbol") == 0) { - value = PyObject_GetAttrString(pPythonValue,"value"); - out = YCPSymbol(PyString_AsString(value)); - return true; - } - if (type.compare("path") == 0) { - value = PyObject_GetAttrString(pPythonValue,"value"); - out = YCPPath(PyString_AsString(value)); - return true; - } - } //end of if (PyString_Check(py_attr)) - } //end of if (PyObject_HasAttrString(pPythonValue, "type")) - } - return false; -} - - /** - * Transfer from ycp simple type to python simple type - * transfered are: boolean, integer, string, float, - **/ -PyObject* YPython::YCPTypeToPythonSimpleType(YCPValue in) { - - if (in.isNull()) { - return Py_None; - - } else if (in->isBoolean()) { - long val = 0; - bool true_false = in->asBoolean()->value(); - true_false ? val = 1 : val =0; - return PyBool_FromLong(val); - - } else if (in->isInteger()) { - return PyInt_FromLong(in->asInteger()->value()); - - } else if (in->isFloat()) { - double val = in->asFloat()->value(); - return PyFloat_FromDouble(val); - - } else if (in->isString()) { - return PyString_FromString((in->asString()->value()).c_str()); - - } else if (in->isVoid()) { - return Py_None; - - } else if (in->isPath()) { - if (pPathClass) { - PyObject *value = PyString_FromString(in->asPath()->toString().c_str()); - PyObject *pyArgs = PyTuple_New(1); - PyTuple_SetItem(pyArgs, 0, value); - return PyInstance_New(pPathClass, pyArgs, NULL); + pMain = PyImport_AddModule("__main__"); + //PyRun_SimpleString("import __main__"); + pMainDict = PyModule_GetDict(pMain); + pFunc = PyDict_GetItemString(pMainDict, function.c_str()); + pArgs = PyTuple_New(argList->size()-1); - } else { - return Py_None; - } - } else if (in->isSymbol()) { - if (pSymbolClass) { - //add hack delete first char (`) from output - PyObject *value = PyString_FromString(in->asSymbol()->toString().erase(0,1).c_str()); - PyObject *pyArgs = PyTuple_New(1); - PyTuple_SetItem(pyArgs, 0, value); - return PyInstance_New(pSymbolClass, pyArgs, NULL); + //Parsing argumments + PyObject *pArg; + y2milestone ("name of function %s and number of arguments %d", function.c_str(), argList->size()-1); + for ( int i=1; i < argList->size(); i++ ) { + pArg = YCPTypeToPythonType(argList->value(i)); + PyTuple_SetItem(pArgs,i-1, pArg); + } + + pReturn = PyObject_CallObject(pFunc, pArgs); + + Py_CLEAR(pArgs); + + if (pReturn){ + result = PythonTypeToYCPType(pReturn); // create YCP value + }else + y2error("pReturn == 0"); + Py_CLEAR(pReturn); + + if (result.isNull ()) { + y2error ("Result is NULL when returning from %s", function.c_str()); + result = YCPVoid (); + } + + return result; +} + + +YCPValue YPython::PythonTypeToYCPType(PyObject *pythonValue) +{ + // null (in python None) + if (pythonValue == Py_None) + return YCPNull(); + + //boolean + if (PyBool_Check(pythonValue)){ + int cmp_result = PyObject_Compare(pythonValue, Py_True); + if (PyErr_Occurred() != NULL){ + return YCPNull(); + } + if (cmp_result == 0){ + return YCPBoolean(true); + }else{ + return YCPBoolean(false); + } + } + + // integer + if (PyInt_Check(pythonValue)) + return YCPInteger(PyInt_AsLong(pythonValue)); + + //float + if (PyFloat_Check(pythonValue)) + return YCPFloat(PyFloat_AsDouble(pythonValue)); + + //string + if (PyString_Check(pythonValue)) + return YCPString(PyString_AsString(pythonValue)); + + //list + if (PyList_Check(pythonValue)) + return fromPythonListToYCPList(pythonValue); + + //tuple -> list + if (PyTuple_Check(pythonValue)) + return fromPythonTupleToYCPList(pythonValue); + + //dict -> map + if (PyDict_Check(pythonValue)) + return fromPythonDictToYCPMap(pythonValue); - } else { + // term, symbol, path + switch (getYCPType(pythonValue)){ + case SYMBOL: + return YCPSymbol(Symbol_getValue((Symbol *)pythonValue)); + case PATH: + return YCPPath(Path_getValue((Path *)pythonValue)); + case TERM: + return fromPythonTermToYCPTerm(pythonValue); + case NOT_YCP_TYPE: + return YCPNull(); + } + + return YCPNull(); +} + + +PyObject *YPython::YCPTypeToPythonType(YCPValue value) +{ + // null -> None + if (value.isNull()) + return Py_None; + //void -> None + if (value->isVoid()) return Py_None; - } - } else - return NULL; + + // boolean + if (value->isBoolean()){ + if (value->asBoolean()->value()) + return PyBool_FromLong(1); + return PyBool_FromLong(0); + } + + //integer + if (value->isInteger()) + return PyInt_FromLong(value->asInteger()->value()); + + //float + if (value->isFloat()) + return PyFloat_FromDouble(value->asFloat()->value()); + + //string + if (value->isString()) + return PyString_FromString(value->asString()->value().c_str()); + + //list -> tuple + if (value->isList()) + return fromYCPListToPythonTuple(value->asList()); + + //map -> dict + if (value->isMap()) + return fromYCPMapToPythonDict(value->asMap()); + + //path + if (value->isPath()) + return Path_NewString(value->asPath()->toString().c_str()); + + //symbol + if (value->isSymbol()) + return Symbol_NewString(value->asSymbol()->toString().erase(0,1).c_str()); + + //term + if (value->isTerm()) + return fromYCPTermToPythonTerm(value->asTerm()); + + return Py_None; } + + + + +/********** PRIVATE: **********/ + /** * Convert a Python list to a YCPList. **/ - YCPList YPython::fromPythonListToYCPList (PyObject* pPythonList) { - YCPList ycp_List; - PyObject * pItem; - YCPValue ycp_value; - //checking for list (is it really list?) - if (PyList_Check(pPythonList) > 0) { - int list_size = PyList_Size(pPythonList); - for (int i = 0; i < list_size; i++) { - pItem = PyList_GetItem(pPythonList, i); - if (PythonTypeToYCPSimpleType(pItem, ycp_value)) { + YCPList ycp_List; + PyObject * pItem; + YCPValue ycp_value; + + //checking for list (is it really list?) + if (PyList_Check(pPythonList) > 0) { + int list_size = PyList_Size(pPythonList); + for (int i = 0; i < list_size; i++) { + pItem = PyList_GetItem(pPythonList, i); + ycp_value = PythonTypeToYCPType(pItem); + if (!ycp_value.isNull ()) { - ycp_List->add(ycp_value); + ycp_List->add(ycp_value); } else { - return YCPNull (); + return YCPNull (); } - } else { //end of if (PythonTypeToYCPSimpleType(item, ycp_value)) - ycp_value = fromPythonListToYCPList(pItem); - if (ycp_value.isNull ()) - ycp_value = fromPythonDictToYCPMap (pItem); - if (ycp_value.isNull ()) - ycp_value = fromPythonTupleToYCPList(pItem); - if (ycp_value.isNull ()) - ycp_value = fromPythonTermToYCPTerm(pItem); - - if (!ycp_value.isNull ()) - ycp_List->add(ycp_value); - else - return YCPNull (); - } // end of else for if (PythonTypeToYCPSimpleType(item, ycp_value)) - } //end of for (int i = 0; i < list_size; i++) - } else { //end if (PyList_Check(pPythonList) > 0) - y2milestone ("Value is not Python List"); - return YCPNull (); - } //end else of if (PyList_Check(pPythonList) > 0) + } + }else{ + y2milestone ("Value is not Python List"); + return YCPNull (); + } - return ycp_List; + return ycp_List; } /** * Convert a Python Tuple to a YCPList. **/ YCPList YPython::fromPythonTupleToYCPList (PyObject* pPythonTuple) { - YCPList ycp_List; - PyObject * pItem; - YCPValue ycp_value; - //checking for list (is it really list?) - if (PyTuple_Check(pPythonTuple) > 0) { - int list_size = PyTuple_Size(pPythonTuple); - for (int i = 0; i < list_size; i++) { - pItem = PyTuple_GetItem(pPythonTuple, i); - if (PythonTypeToYCPSimpleType(pItem, ycp_value)) { + YCPList ycp_List; + PyObject * pItem; + YCPValue ycp_value; + + //checking for list (is it really list?) + if (PyTuple_Check(pPythonTuple) > 0) { + int list_size = PyTuple_Size(pPythonTuple); + for (int i = 0; i < list_size; i++) { + pItem = PyTuple_GetItem(pPythonTuple, i); + ycp_value = PythonTypeToYCPType(pItem); + if (!ycp_value.isNull ()) { - ycp_List->add(ycp_value); + ycp_List->add(ycp_value); } else { - return YCPNull (); - } - } else { //end of if (PythonTypeToYCPSimpleType(item, ycp_value)) - ycp_value = fromPythonListToYCPList(pItem); - - if (ycp_value.isNull ()) - ycp_value = fromPythonTupleToYCPList(pItem); - - if (ycp_value.isNull ()) - ycp_value = fromPythonDictToYCPMap(pItem); - - if (ycp_value.isNull ()) - ycp_value = fromPythonTermToYCPTerm(pItem); - - if (!ycp_value.isNull ()) - ycp_List->add(ycp_value); - else return YCPNull (); - } // end of else for if (PythonTypeToYCPSimpleType(item, ycp_value)) - } //end of for (int i = 0; i < list_size; i++) - } else { //end if (PyTuple_Check(pPythonList) > 0) - y2milestone ("Value is not Python Tuple"); - return YCPNull (); - } //end else of if (PyTuple_Check(pPythonList) > 0) + } + } + }else{ + y2milestone ("Value is not Python Tuple"); + return YCPNull (); + } - return ycp_List; + return ycp_List; } /** * Convert a YCPList to a Python list. **/ PyObject* YPython::fromYCPListToPythonList (YCPValue ycp_List) { + PyObject* pPythonList; + PyObject* pItem; + int ret = 0; - PyObject* pPythonList; - PyObject* pItem; - int ret = 0; - if (ycp_List->isList()) { - if (ycp_List->asList()->size()>0) + if (ycp_List->isList()) { pPythonList = PyList_New(ycp_List->asList()->size()); - else { - y2error("YCP list is empty."); - return Py_None; - } - y2milestone ("Size of list %d",ycp_List->asList()->size()); - for ( int i = 0; i < ycp_List->asList()->size(); i++ ) { - pItem = YCPTypeToPythonSimpleType(ycp_List->asList()->value(i)); - if (!pItem) - pItem = fromYCPListToPythonList(ycp_List->asList()->value(i)); + y2milestone ("Size of list %d",ycp_List->asList()->size()); + for ( int i = 0; i < ycp_List->asList()->size(); i++ ) { + pItem = YCPTypeToPythonType(ycp_List->asList()->value(i)); + ret = PyList_SetItem(pPythonList, i, pItem); - if (!pItem) - pItem = fromYCPListToPythonTuple(ycp_List->asList()->value(i)); - - if (!pItem) - pItem = fromYCPMapToPythonDict(ycp_List->asList()->value(i)); - - if (!pItem) - pItem = fromYCPTermToPythonTerm(ycp_List->asList()->value(i)); - - ret = PyList_SetItem(pPythonList, i, pItem); - - if (ret <0) - y2error("PyList_SetItem doesn't add item into python list."); - - } - Py_INCREF(pPythonList); - return pPythonList; + if (ret <0) + y2error("PyList_SetItem doesn't add item into python list."); + } + Py_INCREF(pPythonList); //TODO: Review this - is it really needed? PyList_New returns New Reference. + return pPythonList; - } else { - y2milestone ("Value is not YCPList"); - return NULL; - } + } else { + y2milestone ("Value is not YCPList"); + return Py_None; + } } /** * Convert a YCPList to a Python tuple. **/ PyObject* YPython::fromYCPListToPythonTuple (YCPValue ycp_List) { + PyObject* pPythonTuple; + PyObject* pItem; + int ret = 0; - PyObject* pPythonTuple; - PyObject* pItem; - int ret = 0; - if (ycp_List->isList()) { - if (ycp_List->asList()->size()>0) + if (ycp_List->isList()) { pPythonTuple = PyTuple_New(ycp_List->asList()->size()); - else { - y2error("YCP list is empty."); - return Py_None; - } - y2milestone ("Size of list %d",ycp_List->asList()->size()); - for ( int i = 0; i < ycp_List->asList()->size(); i++ ) { - pItem = YCPTypeToPythonSimpleType(ycp_List->asList()->value(i)); - - if (!pItem) - pItem = fromYCPListToPythonList(ycp_List->asList()->value(i)); - - if (!pItem) - pItem = fromYCPListToPythonTuple(ycp_List->asList()->value(i)); - - if (!pItem) - pItem = fromYCPMapToPythonDict(ycp_List->asList()->value(i)); - - if (!pItem) - pItem = fromYCPTermToPythonTerm(ycp_List->asList()->value(i)); - ret = PyTuple_SetItem(pPythonTuple, i, pItem); + y2milestone ("Size of list %d",ycp_List->asList()->size()); + for ( int i = 0; i < ycp_List->asList()->size(); i++ ) { + pItem = YCPTypeToPythonType(ycp_List->asList()->value(i)); + ret = PyTuple_SetItem(pPythonTuple, i, pItem); - if (ret <0) - y2error("PyList_SetItem doesn't add item into python list."); - - } - Py_INCREF(pPythonTuple); - return pPythonTuple; + if (ret <0) + y2error("PyList_SetItem doesn't add item into python list."); + } + Py_INCREF(pPythonTuple); //TODO: Review this. + return pPythonTuple; - } else { - y2milestone ("Value is not YCPList"); - return NULL; - } + } else { + y2milestone ("Value is not YCPList"); + return NULL; + } } /** * Convert a YCPMap to a Python Dictionary. + * If something goes wrong, iteration is trying proceed... **/ PyObject* YPython::fromYCPMapToPythonDict (YCPValue ycp_Map) { + PyObject* pPythonDict; + PyObject* pKey; + PyObject* pValue; + int ret = -1; - PyObject* pPythonDict; - PyObject* pKey; - PyObject* pValue; - int ret = -1; - if (ycp_Map->isMap()) { - if (ycp_Map->asMap()->size()>0) { + if (ycp_Map->isMap()) { pPythonDict = PyDict_New(); + for (YCPMapIterator it = ycp_Map->asMap()->begin(); it != ycp_Map->asMap()->end(); ++it ) { - pKey = YCPTypeToPythonSimpleType(it.key()); - if (!pKey) { - y2error("Transfer key YCPMap to PyObject falsed."); - return NULL; - } - pValue = YCPTypeToPythonSimpleType(it.value()); - if (!pValue) - pValue = fromYCPListToPythonList(it.value()); - if (!pValue) - pValue = fromYCPListToPythonTuple(it.value()); - if (!pValue) - pValue = fromYCPMapToPythonDict(it.value()); - if (!pValue) - pValue = fromYCPTermToPythonTerm(it.value()); - - - if (pValue) { - ret = PyDict_SetItem(pPythonDict, pKey, pValue); - if (ret < 0) { - y2error("Adding value and key from YCPMap to Python Dictionary falsed."); - return NULL; - } - } else { - y2error("Transfer value from YCPMap to PyObject falsed."); - return NULL; - - } - } // end of for (YCPMapIterator it = ycp_Map->begin(); it != ycp_Map->end(); ++it ) + pKey = YCPTypeToPythonType(it.key()); + pValue = YCPTypeToPythonType(it.value()); - } else { // end if (ycp_Map->asMap()->size()>0) - y2error("YCP map is empty."); - return NULL; - } // end else of if (ycp_Map->asMap()->size()>0) + if (pValue && pKey){ + ret = PyDict_SetItem(pPythonDict, pKey, pValue); + + if (ret < 0) + y2error("Adding value and key from YCPMap to Python Dictionary falsed."); + }else{ + y2error("Transformation key and/or value to python type failed."); + } + } + }else{ + y2milestone ("Value is not YCPMap"); + return Py_None; + } - } else { // end if (ycp_Map->isMap()) - y2milestone ("Value is not YCPMap"); - return NULL; - } // end else of if (ycp_Map->isMap()) - Py_INCREF(pPythonDict); - return pPythonDict; + Py_INCREF(pPythonDict);// TODO: Review this + return pPythonDict; } @@ -563,140 +423,75 @@ * Convert a Python Dictionary to a YCPMap. **/ YCPMap YPython::fromPythonDictToYCPMap (PyObject* pPythonDict) { + YCPValue ycp_key; + YCPValue ycp_value; + YCPMap ycp_Map; + + if (PyDict_Check(pPythonDict)>0) { + if (PyDict_Size(pPythonDict) == 0) + return ycp_Map; + + PyObject *key, *value; + Py_ssize_t pos = 0; + + while (PyDict_Next(pPythonDict, &pos, &key, &value)) { + ycp_key = PythonTypeToYCPType(key); + ycp_value = PythonTypeToYCPType(value); - PyObject* pListKeys; - PyObject* pListValues; - YCPValue ycp_key; - YCPValue ycp_value; - YCPMap ycp_Map; - if (PyDict_Check(pPythonDict)>0) { - if (PyDict_Size(pPythonDict)>0) { - pListKeys = PyDict_Keys(pPythonDict); - pListValues = PyDict_Values(pPythonDict); - for (int i=0; i < PyList_Size(pListKeys); i++) { - if (PythonTypeToYCPSimpleType(PyList_GetItem(pListKeys, i),ycp_key)) { - if (PythonTypeToYCPSimpleType(PyList_GetItem(pListValues, i),ycp_value)) { - ycp_Map->add (ycp_key, ycp_value); - } else { - ycp_value = fromPythonListToYCPList(PyList_GetItem(pListValues, i)); - if (ycp_value.isNull ()) - ycp_value = fromPythonTupleToYCPList(PyList_GetItem(pListValues, i)); - if (ycp_value.isNull ()) - ycp_value = fromPythonDictToYCPMap (PyList_GetItem(pListValues, i)); - if (ycp_value.isNull ()) - ycp_value = fromPythonTermToYCPTerm(PyList_GetItem(pListValues, i)); - - ycp_Map->add (ycp_key, ycp_value); - } + ycp_Map->add(ycp_key, ycp_value); + } + }else{ + y2milestone ("Value is not python dictionary"); + return YCPNull(); + } - } else { - y2error("Cannot convert key from python dictionary"); - return YCPNull (); - } - } - - } else { - y2milestone ("Python dictionary is empty"); - return YCPNull (); - } - } else { - y2milestone ("Value is not python dictionary"); - return YCPNull (); - } - return ycp_Map; + return ycp_Map; } + /** * Convert a Python Term to a YCPTerm. + * Argument should be Term! **/ +YCPTerm YPython::fromPythonTermToYCPTerm (PyObject* pythonTerm) { + PyObject *value; + string name; + YCPValue ycp_value; + + if (!isTerm(pythonTerm)){ + y2error("Argument is not Term!"); + return YCPNull(); + } + + name = Term_getName((Term *)pythonTerm); + value = Term_getValue((Term *)pythonTerm); + ycp_value = fromPythonTupleToYCPList(value); -YCPTerm YPython::fromPythonTermToYCPTerm (PyObject* pPythonTerm) { - PyObject *value; - PyObject *pyName; - string name; - YCPValue ycp_value; - - if (PyInstance_Check(pPythonTerm)) { - if (PyObject_HasAttrString(pPythonTerm, "type")) { - PyObject * py_attr = PyObject_GetAttrString(pPythonTerm,"type"); - if (PyString_Check(py_attr)) { - string type = PyString_AsString(py_attr); - if (type.compare("term") == 0) { - pyName = PyObject_GetAttrString(pPythonTerm,"name"); - if (PyString_Check(pyName)) { - name = PyString_AsString(pyName); - value = PyObject_GetAttrString(pPythonTerm,"value"); - if (PyTuple_Check(value)) - - ycp_value = fromPythonTupleToYCPList(value); - - if (!ycp_value.isNull ()) { - //printf("term is: %s\n",YCPTerm(name,ycp_value->asList())->toString().c_str()); - return YCPTerm(name,ycp_value->asList()); - - } else { - y2error("Value was not converted to YCPTerm"); - return YCPNull (); - } - } else { - y2error ("The name of Term is not string"); - return YCPNull (); - } - } else { - y2milestone ("Value is not python Term"); - return YCPNull (); - } - } else { - y2error ("Value is not any python class for ycp types"); - return YCPNull (); - } - } - } else { - return YCPNull (); - } - return YCPNull (); + if (!ycp_value.isNull()) + return YCPTerm(name, ycp_value->asList()); + + return YCPNull(); } /** * Convert a YCPTerm to a Python Term. + * Given argument should be term! **/ PyObject* YPython::fromYCPTermToPythonTerm (YCPValue ycp_Term) { + PyObject *value; - PyObject* pPythonTerm; - PyObject* pName; - PyObject* pValue; - - if (ycp_Term->isTerm()) { - if (ycp_Term->asTerm()->args()->size()>0) { - pName = PyString_FromString(ycp_Term->asTerm()->name().c_str()); - YCPValue list = ycp_Term->asTerm()->args(); - pValue = fromYCPListToPythonTuple(list); - if (pValue) { - if (pTermClass) { - PyObject *pyArgs = PyTuple_New(2); - PyTuple_SetItem(pyArgs, 0, pName); - PyTuple_SetItem(pyArgs, 1, pValue); - pPythonTerm = PyInstance_New(pTermClass, pyArgs, NULL); - } else { - y2error("There is missing Term class in python"); - return Py_None; - } - } else { - y2error("Converting to python tuple failed"); - return Py_None; - } - - } else { // end if (ycp_Map->asMap()->size()>0) - y2error("YCP term is empty."); - return NULL; - } // end else of if (ycp_Map->asMap()->size()>0) + if (!ycp_Term->isTerm()){ + y2error("Argument is not term!"); + return Py_None; + } + + value = fromYCPListToPythonTuple(ycp_Term->asTerm()->args()); + if (value == Py_None){ + y2error("fromYCPListToPythonTuple FIALED"); + return Py_None; + } - } else { // end if (ycp_Map->isMap()) - y2milestone ("Value is not YCPTerm"); - return NULL; - } // end else of if (ycp_Map->isMap()) - Py_INCREF(pPythonTerm); - return pPythonTerm; + return Term_NewString(ycp_Term->asTerm()->name().c_str(), value); } Modified: trunk/python-bindings/src/YPython.h URL: http://svn.opensuse.org/viewcvs/yast/trunk/python-bindings/src/YPython.h?re… ============================================================================== --- trunk/python-bindings/src/YPython.h (original) +++ trunk/python-bindings/src/YPython.h Thu Sep 27 15:34:02 2007 @@ -63,18 +63,20 @@ static YPython * _yPython; - /** - * Transfer from python simple type to ycp simple type - * transfered are: boolean, integer, string, float, - **/ - bool PythonTypeToYCPSimpleType(PyObject* pPythonValue, YCPValue &out); - - /** - * Transfer from ycp simple type to python simple type - * transfered are: boolean, integer, string, float, - **/ - PyObject* YCPTypeToPythonSimpleType(YCPValue in); + /** + * Transform Python type to YCP type and return new YCPValue built + * from python object. + */ + YCPValue PythonTypeToYCPType(PyObject*); + + /** + * Transform YCP type to Python type and return reference to new + * Python object. + */ + PyObject *YCPTypeToPythonType(YCPValue); + +private: /** * Convert a Python list to a YCPList. **/ @@ -116,12 +118,6 @@ **/ PyObject* fromYCPTermToPythonTerm (YCPValue ycp_Term); - - PyObject *pPathClass; - PyObject *pSymbolClass; - PyObject *pTermClass; - - protected: /** -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41141 - in /branches/tmp/sh/mod-ui/ncurses/src: NCAskForFile.cc NCPopupTextEntry.cc NCPopupTextEntry.h README_mod-ui pkg/NCPkgPopupDeps.cc
by gs＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: gs Date: Thu Sep 27 15:31:52 2007 New Revision: 41141 URL: http://svn.opensuse.org/viewcvs/yast?rev=41141&view=rev Log: use setValue()/value() Modified: branches/tmp/sh/mod-ui/ncurses/src/NCAskForFile.cc branches/tmp/sh/mod-ui/ncurses/src/NCPopupTextEntry.cc branches/tmp/sh/mod-ui/ncurses/src/NCPopupTextEntry.h branches/tmp/sh/mod-ui/ncurses/src/README_mod-ui branches/tmp/sh/mod-ui/ncurses/src/pkg/NCPkgPopupDeps.cc Modified: branches/tmp/sh/mod-ui/ncurses/src/NCAskForFile.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCA… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCAskForFile.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCAskForFile.cc Thu Sep 27 15:31:52 2007 @@ -209,7 +209,9 @@ fileName = new NCInputField( hSplit2, // label for text field showing the filename _( "&File name:" ), - false ); // passWordMode = false + false, + 100, + 50 ); // passWordMode = false fileName->setValue( iniFileName ); fileName->setInputMaxLength( 100 ); // FIXME @@ -325,7 +327,7 @@ fileList->fillList( ); if ( iniFileName == "" ) // show the currently selected file - fileName->setText( fileList->getCurrentFile() ); + fileName->setValue( fileList->getCurrentFile() ); } /////////////////////////////////////////////////////////////////// @@ -351,7 +353,7 @@ else if ( postevent.keySymbol == "CursorRight" ) { fileList->setKeyboardFocus(); - fileName->setText( fileList->getCurrentFile() ); + fileName->setValue( fileList->getCurrentFile() ); return true; } @@ -415,7 +417,8 @@ { if ( !postevent.result.isNull() ) { - fileName->setText( postevent.result->asString() ); + // FIXME - check this !!! + fileName->setValue( postevent.result->asString()->value() ); } } else @@ -483,10 +486,10 @@ // string NCAskForExistingFile::getFileName() { - if ( fileName->getText()->value() == "" ) + if ( fileName->value() == "" ) return fileList->getCurrentFile(); else - return fileName->getText()->value(); + return fileName->value(); } /////////////////////////////////////////////////////////////////// @@ -519,5 +522,5 @@ // string NCAskForSaveFileName::getFileName() { - return fileName->getText()->value(); + return fileName->value(); } Modified: branches/tmp/sh/mod-ui/ncurses/src/NCPopupTextEntry.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCP… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCPopupTextEntry.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCPopupTextEntry.cc Thu Sep 27 15:31:52 2007 @@ -28,8 +28,8 @@ // DESCRIPTION : // NCPopupTextEntry::NCPopupTextEntry( const wpos at, - const YCPString & label, - const YCPString & text, + const string & label, + const string & text, unsigned maxInput, unsigned maxFld, NCInputField::FTYPE t ) @@ -39,12 +39,15 @@ YWidgetOpt opt; //opt.notifyMode.setValue( true ); wtext = new NCInputField( this, - label->toString(), - false // passwordMode = false + label, + false, // passwordMode = false + maxInput, + maxFld ); wtext->setFldtype( t ); wtext->setReturnOnReturn( true ); - addChild( wtext ); + wtext->setNotify( true ); + //addChild( wtext ); } /////////////////////////////////////////////////////////////////// Modified: branches/tmp/sh/mod-ui/ncurses/src/NCPopupTextEntry.h URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCP… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCPopupTextEntry.h (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCPopupTextEntry.h Thu Sep 27 15:31:52 2007 @@ -45,16 +45,19 @@ public: NCPopupTextEntry( const wpos at, - const YCPString & label, - const YCPString & text, + const string & label, + const string & text, unsigned maxInput = 0, unsigned maxFld = 0, NCInputField::FTYPE t = NCInputField::PLAIN ); virtual ~NCPopupTextEntry(); - void setText( const YCPString & ntext ) { wtext->setText( ntext ); } - YCPString getText() { return wtext->getText(); } + void setValue( const string & ntext ) { wtext->setValue( ntext ); } + string value() { return wtext->value(); } + + //void setText( const YCPString & ntext ) { wtext->setText( ntext ); } + //YCPString getText() { return wtext->getText(); } }; /////////////////////////////////////////////////////////////////// Modified: branches/tmp/sh/mod-ui/ncurses/src/README_mod-ui URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/REA… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/README_mod-ui (original) +++ branches/tmp/sh/mod-ui/ncurses/src/README_mod-ui Thu Sep 27 15:31:52 2007 @@ -21,7 +21,8 @@ selectItem() added TODO: check selectItem() methods ! - YItem::index() is always -1 ??? (see debug in NCTree.cc, line 343) - - are the methods complete (methods from YTree and myPad() methods called correctly) + - are the methods complete and are the methods from YTree and myPad() + called correctly? wHandleInput() TODO: (oldCurrentItem != getCurrentItem()) doesn't return true for different items @@ -30,7 +31,8 @@ EVENTs ------ -remove YCPValue result; from NCursesEvent +TODO: remove YCPValue result; from NCursesEvent (check where 'result' is used, + maybe a string value can be used) RadioButtonGroup @@ -43,12 +45,48 @@ selectItem() added addItem (string, bool ) added -TO DO: test NCPkgPopupDeps which uses addItem +TODO: test NCPkgPopupDeps which uses addItem(string, bool) NCComboBox ---------- -listPopup(): Don't create widget NCPopupList on the stack!!! This leads to segfault when widgets are deleted from YUI at the end! call YDialog::deleteTopmostDialog(); to delete NCPopupList + +NCPopupMenu/NCMenuButton +------------------------- +Don't create NCPopupMenu on the stack - reason see above + +NCMenuButton +------------- +TODO: selection set correctly? + +Don't create NCAskForDirectory/NCAskForExistingFile/NCAskForExistingDirectory +on the stack (->YNCursesUI.cc) + +NCFrame +------- +whether a frame is strechable or not depends on the children, if a children is +stretchable, the frame also is stretchable +-> NCAskForDirectory/NCAskForFile: setStretchable for NCComboBox true + +NCInputField +------------ +TODO: what's the difference between InputMaxLength and maxInputLength ??? + +NCIntField +---------- +Don't create NCPopupTextEntry on the stack !!! + +NCPopupTextEntry +---------------- +YCP... removed + +NCDialog +-------- +TODO: Check whether NCPopupInfo is created/deleted correctly (YDialog::deleteTopmostDialog() missing or not?) + +NCPopupInfo +----------- +check NCPopupInfo (F1) Modified: branches/tmp/sh/mod-ui/ncurses/src/pkg/NCPkgPopupDeps.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/pkg… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/pkg/NCPkgPopupDeps.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/pkg/NCPkgPopupDeps.cc Thu Sep 27 15:31:52 2007 @@ -306,7 +306,7 @@ zypp::ResolverProblem_Ptr problem = problems[index].first; zypp::ProblemSolution_Ptr user_solution = problems[index].second; - details->setText( YCPString(problem->details()) ); + details->setValue( problem->details() ); details->setCurPos( 0 ); zypp::ProblemSolutionList solutions = problem->solutions (); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41140 - /branches/tmp/sh/mod-ui/ncurses/src/NCIntField.cc
by gs＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: gs Date: Thu Sep 27 15:31:19 2007 New Revision: 41140 URL: http://svn.opensuse.org/viewcvs/yast?rev=41140&view=rev Log: create NCPopupTextEntry with 'new' Modified: branches/tmp/sh/mod-ui/ncurses/src/NCIntField.cc Modified: branches/tmp/sh/mod-ui/ncurses/src/NCIntField.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCI… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCIntField.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCIntField.cc Thu Sep 27 15:31:19 2007 @@ -414,21 +414,24 @@ + "," + numstring( maxValue() ) + "]" ); string text( 1, (char )first); - NCPopupTextEntry dialog( at, label, text, vlen, 0, - NCInputField::NUMBER ); + NCPopupTextEntry * dialog = new NCPopupTextEntry( at, label, text, vlen, 0, + NCInputField::NUMBER ); - while ( dialog.post() != -1 ) { + while ( dialog->post() != -1 ) { - int nval = atoi( dialog.getText()->value().c_str() ); + int nval = atoi( dialog->value().c_str() ); if ( nval < minValue() ) { - dialog.setText( numstring( minValue() ) ); + dialog->setValue( numstring( minValue() ) ); } else if ( maxValue() < nval ) { - dialog.setText( numstring( maxValue() ) ); + dialog->setValue( numstring( maxValue() ) ); } else { setValue( nval ); break; } ::beep(); } + + YDialog::deleteTopmostDialog(); + return 0; } -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41139 - /branches/tmp/sh/mod-ui/ncurses/src/ncursesw.cc
by gs＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: gs Date: Thu Sep 27 15:29:27 2007 New Revision: 41139 URL: http://svn.opensuse.org/viewcvs/yast?rev=41139&view=rev Log: debug added Modified: branches/tmp/sh/mod-ui/ncurses/src/ncursesw.cc Modified: branches/tmp/sh/mod-ui/ncurses/src/ncursesw.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/ncu… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/ncursesw.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/ncursesw.cc Thu Sep 27 15:29:27 2007 @@ -277,6 +277,8 @@ if ( cols + begin_x > NCursesWindow::cols() ) cols = NCursesWindow::cols() - begin_x; + WIDDBG << "Lines: " << lines << " Cols: " << cols << " y: " << begin_y << " x: " << begin_x << endl; + w = ::newwin(lines, cols, begin_y, begin_x); if (w == 0) { err_handler("Cannot construct window"); @@ -328,7 +330,7 @@ // Friendly enough, this also works for pads. w = ::derwin(win.w, l, c, begin_y, begin_x); if (w == 0) { - //WIDERR << "Throw " << wpos(begin_y, begin_x) << wsze(l, c) << endl; + WIDERR << "Throw " << wpos(begin_y, begin_x) << wsze(l, c) << endl; err_handler("Cannot construct subwindow"); } //WIDMIL << "created " << wpos(begin_y, begin_x) << wsze(l, c) << endl; -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0

[yast-commit] r41138 - /branches/tmp/sh/mod-ui/ncurses/src/NCComboBox.cc
by gs＠svn.opensuse.org 27 Sep '07

27 Sep '07

Author: gs Date: Thu Sep 27 15:25:27 2007 New Revision: 41138 URL: http://svn.opensuse.org/viewcvs/yast?rev=41138&view=rev Log: add comment Modified: branches/tmp/sh/mod-ui/ncurses/src/NCComboBox.cc Modified: branches/tmp/sh/mod-ui/ncurses/src/NCComboBox.cc URL: http://svn.opensuse.org/viewcvs/yast/branches/tmp/sh/mod-ui/ncurses/src/NCC… ============================================================================== --- branches/tmp/sh/mod-ui/ncurses/src/NCComboBox.cc (original) +++ branches/tmp/sh/mod-ui/ncurses/src/NCComboBox.cc Thu Sep 27 15:25:27 2007 @@ -296,6 +296,14 @@ Redraw(); } +/////////////////////////////////////////////////////////////////// +// +// +// METHOD NAME : NCComboBox::setText +// METHOD TYPE : void +// +// DESCRIPTION : +// void NCComboBox::setText( const string & ntext ) { privText = NCstring( ntext ); -- To unsubscribe, e-mail: yast-commit+unsubscribe(a)opensuse.org For additional commands, e-mail: yast-commit+help(a)opensuse.org

1 0